[regexp] Move RegExp.prototype.test to TF

[regexp] Move RegExp.prototype.test to TF

This results in a speedup of around 2x. RegExpExec is also ported in
this CL.

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

[jgruber, 2016-10-21 10:48:06]

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

[commit-bot: I haz the power, 2016-10-21 10:48:12]

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

[jgruber, 2016-10-21 10:52:01]

Description was changed from

==========
[regexp] More RegExp.prototype.test to TF

This results in a speedup of around 2x. RegExpExec is also ported in
this CL.

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

to

==========
[regexp] Move RegExp.prototype.test to TF

This results in a speedup of around 2x. RegExpExec is also ported in
this CL.

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

[jgruber, 2016-10-21 10:52:13]

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

[commit-bot: I haz the power, 2016-10-21 11:17:03]

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

[commit-bot: I haz the power, 2016-10-21 11:17:04]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)

[jgruber, 2016-10-21 14:17:25]

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

[commit-bot: I haz the power, 2016-10-21 14:17:33]

Dry run: 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-10-21 14:32:06]

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

[commit-bot: I haz the power, 2016-10-21 14:32:07]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)

[Yang, 2016-10-24 08:07:06]

lgtm

https://codereview.chromium.org/2441993002/diff/40001/src/builtins/builtins-r...
File src/builtins/builtins-regexp.cc (right):

https://codereview.chromium.org/2441993002/diff/40001/src/builtins/builtins-r...
src/builtins/builtins-regexp.cc:1006:
isolate->factory()->NewStringFromAsciiChecked("exec", TENURED));
Maybe put this in heap-symbols.h?

[jgruber, 2016-10-24 09:27:13]

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

[commit-bot: I haz the power, 2016-10-24 09:27:19]

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

[jgruber, 2016-10-24 09:28:57]

https://codereview.chromium.org/2441993002/diff/40001/src/builtins/builtins-r...
File src/builtins/builtins-regexp.cc (right):

https://codereview.chromium.org/2441993002/diff/40001/src/builtins/builtins-r...
src/builtins/builtins-regexp.cc:1006:
isolate->factory()->NewStringFromAsciiChecked("exec", TENURED));
On 2016/10/24 08:07:06, Yang wrote:
> Maybe put this in heap-symbols.h?

Done.

[jgruber, 2016-10-24 09:36:20]

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

[commit-bot: I haz the power, 2016-10-24 09:36:30]

Dry run: 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-10-24 10:17:29]

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

[commit-bot: I haz the power, 2016-10-24 10:17:30]

Dry run: This issue passed the CQ dry run.

[jgruber, 2016-10-24 12:36:24]

The CQ bit was checked by jgruber@chromium.org

[jgruber, 2016-10-24 12:36:25]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2441993002/#ps80001
(title: "Remove unused variables")

[commit-bot: I haz the power, 2016-10-24 12:36:32]

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-10-24 12:40:52]

Description was changed from

==========
[regexp] Move RegExp.prototype.test to TF

This results in a speedup of around 2x. RegExpExec is also ported in
this CL.

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

to

==========
[regexp] Move RegExp.prototype.test to TF

This results in a speedup of around 2x. RegExpExec is also ported in
this CL.

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

[commit-bot: I haz the power, 2016-10-24 12:40:53]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-11-17 22:11:05]

Description was changed from

==========
[regexp] Move RegExp.prototype.test to TF

This results in a speedup of around 2x. RegExpExec is also ported in
this CL.

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

to

==========
[regexp] Move RegExp.prototype.test to TF

This results in a speedup of around 2x. RegExpExec is also ported in
this CL.

BUG=v8:5339

Committed: https://crrev.com/ae3357d216ed3ce82deed2ad2c6bb526370ef6aa
Cr-Commit-Position: refs/heads/master@{#40532}
==========

[commit-bot: I haz the power, 2016-11-17 22:11:06]

Patchset 5 (id:??) landed as
https://crrev.com/ae3357d216ed3ce82deed2ad2c6bb526370ef6aa
Cr-Commit-Position: refs/heads/master@{#40532}