Skip to content

Commit

Permalink
benchmark: remove v8ForceOptimization calls
Browse files Browse the repository at this point in the history
This removes common.v8ForceOptimization calls from url and vm benchmark
files.

PR-URL: nodejs#11908
Fixes: nodejs#11895
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
  • Loading branch information
lucaslago authored and targos committed Mar 21, 2017
1 parent 5a71cb6 commit 3129ba2
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 5 deletions.
2 changes: 0 additions & 2 deletions benchmark/url/whatwg-url-idna.js
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,6 @@ function main(conf) {
const input = inputs[conf.input][to];
const method = to === 'ascii' ? domainToASCII : domainToUnicode;

common.v8ForceOptimization(method, input);

bench.start();
for (var i = 0; i < n; i++) {
method(input);
Expand Down
2 changes: 0 additions & 2 deletions benchmark/vm/run-in-context.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,6 @@ function main(conf) {

const contextifiedSandbox = vm.createContext();

common.v8ForceOptimization(vm.runInContext,
'0', contextifiedSandbox, options);
bench.start();
for (; i < n; i++)
vm.runInContext('0', contextifiedSandbox, options);
Expand Down
1 change: 0 additions & 1 deletion benchmark/vm/run-in-this-context.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ function main(conf) {

var i = 0;

common.v8ForceOptimization(vm.runInThisContext, '0', options);
bench.start();
for (; i < n; i++)
vm.runInThisContext('0', options);
Expand Down

0 comments on commit 3129ba2

Please sign in to comment.