Skip to content

Commit

Permalink
benchmark: remove querystring forced optimization
Browse files Browse the repository at this point in the history
This removes all instances of %OptimizeFunctionOnNextCall from
querystring benchmarks

PR-URL: #9615
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
  • Loading branch information
bzoz committed Mar 6, 2017
1 parent eba2c62 commit 57b5ce1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 18 deletions.
15 changes: 2 additions & 13 deletions benchmark/querystring/querystring-parse.js
@@ -1,31 +1,20 @@
'use strict';
var common = require('../common.js');
var querystring = require('querystring');
var v8 = require('v8');
var inputs = require('../fixtures/url-inputs.js').searchParams;

var bench = common.createBenchmark(main, {
type: Object.keys(inputs),
n: [1e6],
});

// A deopt followed by a reopt of main() can happen right when the timed loop
// starts, which seems to have a noticeable effect on the benchmark results.
// So we explicitly disable optimization of main() to avoid this potential
// issue.
v8.setFlagsFromString('--allow_natives_syntax');
eval('%NeverOptimizeFunction(main)');

function main(conf) {
var type = conf.type;
var n = conf.n | 0;
var input = inputs[type];
var i;

// Note: we do *not* use OptimizeFunctionOnNextCall() here because currently
// it causes a deopt followed by a reopt, which could make its way into the
// timed loop. Instead, just execute the function a "sufficient" number of
// times before the timed loop to ensure the function is optimized just once.
// Execute the function a "sufficient" number of times before the timed
// loop to ensure the function is optimized just once.
if (type === 'multicharsep') {
for (i = 0; i < n; i += 1)
querystring.parse(input, '&&&&&&&&&&');
Expand Down
5 changes: 0 additions & 5 deletions benchmark/querystring/querystring-stringify.js
@@ -1,7 +1,6 @@
'use strict';
var common = require('../common.js');
var querystring = require('querystring');
var v8 = require('v8');

var bench = common.createBenchmark(main, {
type: ['noencode', 'encodemany', 'encodelast'],
Expand Down Expand Up @@ -36,10 +35,6 @@ function main(conf) {
for (var name in inputs)
querystring.stringify(inputs[name]);

v8.setFlagsFromString('--allow_natives_syntax');
eval('%OptimizeFunctionOnNextCall(querystring.stringify)');
querystring.stringify(input);

bench.start();
for (var i = 0; i < n; i += 1)
querystring.stringify(input);
Expand Down

0 comments on commit 57b5ce1

Please sign in to comment.