[regexp] Add stub for RegExpExec instead of inlining

[regexp] Add stub for RegExpExec instead of inlining

The code produced for RegExpExec is quite large, and we ended up completely
inlining it several spots.  This CL moves RegExpPrototypeExecBody into two
stubs (one each for fast and slow paths) and converts inlined uses into stub
calls. This decreases the local x64 snapshot size by around 80K.

BUG=chromium:688972
Review-Url: https://codereview.chromium.org/2677073004
Cr-Commit-Position: refs/heads/master@{#42965}
Committed: https://chromium.googlesource.com/v8/v8/+/5ea144afe3fd98896fc097a3523de9ab0cb8cd61

[jgruber, 2017-02-06 12:19:53]

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

[commit-bot: I haz the power, 2017-02-06 12:19:56]

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

[jgruber, 2017-02-06 12:44:53]

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

[jgruber, 2017-02-06 12:44:54]

Trying to reduce code size for regexp functions.

If required, we could go further by stubifying RegExpPrototypeBodyWithoutResult
(inlined 4 times).

[Yang, 2017-02-06 12:50:00]

On 2017/02/06 12:44:54, jgruber wrote:
> Trying to reduce code size for regexp functions.
> 
> If required, we could go further by stubifying
RegExpPrototypeBodyWithoutResult
> (inlined 4 times).

lgtm.

[commit-bot: I haz the power, 2017-02-06 12:50:46]

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

[commit-bot: I haz the power, 2017-02-06 12:50:47]

Dry run: This issue passed the CQ dry run.

[jgruber, 2017-02-06 12:54:42]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2017-02-06 12:54:47]

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-02-06 12:56:24]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486385682069330, "parent_rev":
"a495fc92daf0adef64a95e858cd1fe07a430c2e2", "commit_rev":
"5ea144afe3fd98896fc097a3523de9ab0cb8cd61"}

[commit-bot: I haz the power, 2017-02-06 12:56:27]

Description was changed from

==========
[regexp] Add stub for RegExpExec instead of inlining

The code produced for RegExpExec is quite large, and we ended up completely
inlining it several spots.  This CL moves RegExpPrototypeExecBody into two
stubs (one each for fast and slow paths) and converts inlined uses into stub
calls. This decreases the local x64 snapshot size by around 80K.

BUG=chromium:688972
==========

to

==========
[regexp] Add stub for RegExpExec instead of inlining

The code produced for RegExpExec is quite large, and we ended up completely
inlining it several spots.  This CL moves RegExpPrototypeExecBody into two
stubs (one each for fast and slow paths) and converts inlined uses into stub
calls. This decreases the local x64 snapshot size by around 80K.

BUG=chromium:688972

Review-Url: https://codereview.chromium.org/2677073004
Cr-Commit-Position: refs/heads/master@{#42965}
Committed:
https://chromium.googlesource.com/v8/v8/+/5ea144afe3fd98896fc097a3523de9ab0cb...
==========

[commit-bot: I haz the power, 2017-02-06 12:56:29]

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

[jgruber, 2017-02-07 10:07:47]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2679063003/ by jgruber@chromium.org.

The reason for reverting is: Doesn't fix perf regressions in crbug.com/688972
and introduces new ones for RegExp in crbug.com/689395..