[regexp] Don't adapt arguments for @@replace and @@split

[regexp] Don't adapt arguments for @@replace and @@split

Mechanical change to remove argument adaption from RegExp.p.split,
RegExp.p.replace when the actual arguments counts does not match
the formal parameter count (should be a tad faster this way).

BUG=v8:6369
Review-Url: https://codereview.chromium.org/2865313002
Cr-Commit-Position: refs/heads/master@{#45219}
Committed: https://chromium.googlesource.com/v8/v8/+/9fbfd6ead6f7406ca4912610c501146622fa98ea

[jgruber, 2017-05-09 14:52:32]

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

[commit-bot: I haz the power, 2017-05-09 14:52:37]

Dry run: CQ is trying da patch.

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

[jgruber, 2017-05-09 14:52:51]

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

[jgruber, 2017-05-09 14:52:51]

Hey, PTAL!

[gsathya, 2017-05-09 15:05:54]

On 2017/05/09 14:52:51, jgruber wrote:
> Hey, PTAL!

Nice! Can you please update the description with some more text like "Improves
performance since we don't have to create the arguments adaptor"?

LGTM

[commit-bot: I haz the power, 2017-05-09 16:32:40]

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

[commit-bot: I haz the power, 2017-05-09 16:32:41]

Dry run: This issue passed the CQ dry run.

[jgruber, 2017-05-10 06:49:51]

Description was changed from

==========
[regexp] Don't adapt arguments for @@replace and @@split

BUG=v8:6369
==========

to

==========
[regexp] Don't adapt arguments for @@replace and @@split

Mechanical change to remove argument adaption from RegExp.p.split,
RegExp.p.replace when the actual arguments counts does not match
the formal parameter count (should be a tad faster this way).

BUG=v8:6369
==========

[jgruber, 2017-05-10 06:50:09]

On 2017/05/09 15:05:54, gsathya wrote:
> On 2017/05/09 14:52:51, jgruber wrote:
> > Hey, PTAL!
> 
> Nice! Can you please update the description with some more text like "Improves
> performance since we don't have to create the arguments adaptor"?
> 
> LGTM

Thanks, done.

[jgruber, 2017-05-10 06:50:13]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2017-05-10 06:50:24]

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, 2017-05-10 06:51:57]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494399013404650, "parent_rev":
"7cce53586fa45725d3c29e068fb57ab95d067585", "commit_rev":
"9fbfd6ead6f7406ca4912610c501146622fa98ea"}

[commit-bot: I haz the power, 2017-05-10 06:52:04]

Description was changed from

==========
[regexp] Don't adapt arguments for @@replace and @@split

Mechanical change to remove argument adaption from RegExp.p.split,
RegExp.p.replace when the actual arguments counts does not match
the formal parameter count (should be a tad faster this way).

BUG=v8:6369
==========

to

==========
[regexp] Don't adapt arguments for @@replace and @@split

Mechanical change to remove argument adaption from RegExp.p.split,
RegExp.p.replace when the actual arguments counts does not match
the formal parameter count (should be a tad faster this way).

BUG=v8:6369

Review-Url: https://codereview.chromium.org/2865313002
Cr-Commit-Position: refs/heads/master@{#45219}
Committed:
https://chromium.googlesource.com/v8/v8/+/9fbfd6ead6f7406ca4912610c501146622f...
==========

[commit-bot: I haz the power, 2017-05-10 06:52:06]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/9fbfd6ead6f7406ca4912610c501146622f...