From 3129ba2bae718b4b92d69ba55e64c136eccbb7a0 Mon Sep 17 00:00:00 2001 From: Lucas Lago Date: Fri, 17 Mar 2017 17:58:53 -0300 Subject: [PATCH] benchmark: remove v8ForceOptimization calls MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This removes common.v8ForceOptimization calls from url and vm benchmark files. PR-URL: https://github.com/nodejs/node/pull/11908 Fixes: https://github.com/nodejs/node/issues/11895 Reviewed-By: Timothy Gu Reviewed-By: Luigi Pinca Reviewed-By: James M Snell Reviewed-By: Michaël Zasso --- benchmark/url/whatwg-url-idna.js | 2 -- benchmark/vm/run-in-context.js | 2 -- benchmark/vm/run-in-this-context.js | 1 - 3 files changed, 5 deletions(-) diff --git a/benchmark/url/whatwg-url-idna.js b/benchmark/url/whatwg-url-idna.js index 41b4c639de97b6..3d0ea3dc8fe516 100644 --- a/benchmark/url/whatwg-url-idna.js +++ b/benchmark/url/whatwg-url-idna.js @@ -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); diff --git a/benchmark/vm/run-in-context.js b/benchmark/vm/run-in-context.js index 62ebe29146e705..d1d4b5a7abbf20 100644 --- a/benchmark/vm/run-in-context.js +++ b/benchmark/vm/run-in-context.js @@ -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); diff --git a/benchmark/vm/run-in-this-context.js b/benchmark/vm/run-in-this-context.js index f66fd31a1a949e..f3a7e969287d50 100644 --- a/benchmark/vm/run-in-this-context.js +++ b/benchmark/vm/run-in-this-context.js @@ -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);