[js-perf-test] Move SubRegExp class definition outside loop

[js-perf-test] Move SubRegExp class definition outside loop

Defining the subclass within the loop significantly affects subsequent
test results. For instance, the Search benchmark is 50% slower if the
subclass is defined within the loop.

BUG=v8:5339
Committed: https://crrev.com/2f17d5f8e84521140ebd3703e7bb73ef94fc2495
Cr-Commit-Position: refs/heads/master@{#41384}

[jgruber, 2016-11-30 10:23:04]

The CQ bit was checked by jgruber@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-30 10:23:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jgruber, 2016-11-30 10:23:40]

jgruber@chromium.org changed reviewers:
+ cbruni@chromium.org

[jgruber, 2016-11-30 10:32:15]

Description was changed from

==========
[js-perf-test] Move SubRegExp class definition outside loop

Defining the subclass within the loop significantly affects subsequent
test results. For instance, the Search benchmark is 50% slow if the
subclass is defined within the loop.

BUG=v8:5339
==========

to

==========
[js-perf-test] Move SubRegExp class definition outside loop

Defining the subclass within the loop significantly affects subsequent
test results. For instance, the Search benchmark is 50% slower if the
subclass is defined within the loop.

BUG=v8:5339
==========

[commit-bot: I haz the power, 2016-11-30 10:44:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-30 10:44:49]

Dry run: This issue passed the CQ dry run.

[Camillo Bruni, 2016-11-30 12:07:57]

lgtm

[jgruber, 2016-11-30 12:18:18]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2016-11-30 12:18:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-30 12:19:59]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480508298248110, "parent_rev":
"7bb24510a434fb538fac305652fa71deba45b85b", "commit_rev":
"340c629bf830fdabd76059bcc0849f1db842212a"}

[commit-bot: I haz the power, 2016-11-30 12:20:02]

Description was changed from

==========
[js-perf-test] Move SubRegExp class definition outside loop

Defining the subclass within the loop significantly affects subsequent
test results. For instance, the Search benchmark is 50% slower if the
subclass is defined within the loop.

BUG=v8:5339
==========

to

==========
[js-perf-test] Move SubRegExp class definition outside loop

Defining the subclass within the loop significantly affects subsequent
test results. For instance, the Search benchmark is 50% slower if the
subclass is defined within the loop.

BUG=v8:5339
==========

[commit-bot: I haz the power, 2016-11-30 12:20:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-30 12:20:28]

Description was changed from

==========
[js-perf-test] Move SubRegExp class definition outside loop

Defining the subclass within the loop significantly affects subsequent
test results. For instance, the Search benchmark is 50% slower if the
subclass is defined within the loop.

BUG=v8:5339
==========

to

==========
[js-perf-test] Move SubRegExp class definition outside loop

Defining the subclass within the loop significantly affects subsequent
test results. For instance, the Search benchmark is 50% slower if the
subclass is defined within the loop.

BUG=v8:5339

Committed: https://crrev.com/2f17d5f8e84521140ebd3703e7bb73ef94fc2495
Cr-Commit-Position: refs/heads/master@{#41384}
==========

[commit-bot: I haz the power, 2016-11-30 12:20:29]

Patchset 1 (id:??) landed as
https://crrev.com/2f17d5f8e84521140ebd3703e7bb73ef94fc2495
Cr-Commit-Position: refs/heads/master@{#41384}