Skip to content

Commit

Permalink
benchmark: remove forced optimization from es
Browse files Browse the repository at this point in the history
This removes all instances of %OptimizeFunctionOnNextCall from es
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 17c85ff commit ef8cc30
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
4 changes: 0 additions & 4 deletions benchmark/es/defaultparams-bench.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,6 @@ function defaultParams(x = 1, y = 2) {

function runOldStyleDefaults(n) {

common.v8ForceOptimization(oldStyleDefaults);

var i = 0;
bench.start();
for (; i < n; i++)
Expand All @@ -33,8 +31,6 @@ function runOldStyleDefaults(n) {

function runDefaultParams(n) {

common.v8ForceOptimization(defaultParams);

var i = 0;
bench.start();
for (; i < n; i++)
Expand Down
6 changes: 0 additions & 6 deletions benchmark/es/restparams-bench.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,6 @@ function useArguments() {

function runCopyArguments(n) {

common.v8ForceOptimization(copyArguments, 1, 2, 'a', 'b');

var i = 0;
bench.start();
for (; i < n; i++)
Expand All @@ -46,8 +44,6 @@ function runCopyArguments(n) {

function runRestArguments(n) {

common.v8ForceOptimization(restArguments, 1, 2, 'a', 'b');

var i = 0;
bench.start();
for (; i < n; i++)
Expand All @@ -57,8 +53,6 @@ function runRestArguments(n) {

function runUseArguments(n) {

common.v8ForceOptimization(useArguments, 1, 2, 'a', 'b');

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

0 comments on commit ef8cc30

Please sign in to comment.