[bootstrapper] Ensure RegExp constructor has fast properties.

[bootstrapper] Ensure RegExp constructor has fast properties.

Currently x instanceof RegExp checks cannot take the fast path, since
the RegExp constructor has dictionary properties. To avoid that, just
forcibly migrate the RegExp constructor to fast properties again once
it's fully setup in the bootstrapper. This yields a 10x improvement for
x instanceof RegExp checks.

R=yangguo@chromium.org
BUG=v8:5902
Review-Url: https://codereview.chromium.org/2786143004
Cr-Commit-Position: refs/heads/master@{#44316}
Committed: https://chromium.googlesource.com/v8/v8/+/2de2840f2ec238226504fcf05bac6f01cfd2b53d

[Benedikt Meurer, 2017-03-31 17:47:20]

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

[commit-bot: I haz the power, 2017-03-31 17:47:24]

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

[Benedikt Meurer, 2017-03-31 17:48:35]

Hey Yang,

Here's a quickfix for the RegExp constructor. Maybe we can have a more
principled approach for that once we rewrite the bootstrapper (i.e. in a world
w/o JS builtins).
Please take a look.

Thanks,
Benedikt

[Yang, 2017-03-31 18:00:28]

lgtm

[commit-bot: I haz the power, 2017-03-31 18:02:54]

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

[commit-bot: I haz the power, 2017-03-31 18:02:54]

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...)
  v8_linux64_gyp_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng_trigg...)

[Benedikt Meurer, 2017-03-31 18:04:38]

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

[commit-bot: I haz the power, 2017-03-31 18:04:43]

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

[Benedikt Meurer, 2017-03-31 18:04:56]

Description was changed from

==========
[bootstrapper] Ensure RegExp constructor has fast properties.

Currently x instanceof RegExp checks cannot take the fast path, since
the RegExp constructor has dictionary properties. To avoid that, just
forcibly migrate the RegExp constructor to fast properties again once
it's fully setup in the bootstrapper. This yields a 10x improvement for
x instanceof RegExp checks.

R=yangguo@chromium.org
==========

to

==========
[bootstrapper] Ensure RegExp constructor has fast properties.

Currently x instanceof RegExp checks cannot take the fast path, since
the RegExp constructor has dictionary properties. To avoid that, just
forcibly migrate the RegExp constructor to fast properties again once
it's fully setup in the bootstrapper. This yields a 10x improvement for
x instanceof RegExp checks.

R=yangguo@chromium.org
BUG=v8:5902
==========

[Benedikt Meurer, 2017-03-31 18:05:01]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2017-03-31 18:05:06]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2017-03-31 18:05:06]

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/2786143004/#ps20001
(title: "Fix test.")

[commit-bot: I haz the power, 2017-03-31 18:05:14]

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-03-31 18:27:31]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490983506442980,
"parent_rev": "889d205b523c19e34c75770ff045a4c65d530635", "commit_rev":
"2de2840f2ec238226504fcf05bac6f01cfd2b53d"}

[commit-bot: I haz the power, 2017-03-31 18:27:39]

Description was changed from

==========
[bootstrapper] Ensure RegExp constructor has fast properties.

Currently x instanceof RegExp checks cannot take the fast path, since
the RegExp constructor has dictionary properties. To avoid that, just
forcibly migrate the RegExp constructor to fast properties again once
it's fully setup in the bootstrapper. This yields a 10x improvement for
x instanceof RegExp checks.

R=yangguo@chromium.org
BUG=v8:5902
==========

to

==========
[bootstrapper] Ensure RegExp constructor has fast properties.

Currently x instanceof RegExp checks cannot take the fast path, since
the RegExp constructor has dictionary properties. To avoid that, just
forcibly migrate the RegExp constructor to fast properties again once
it's fully setup in the bootstrapper. This yields a 10x improvement for
x instanceof RegExp checks.

R=yangguo@chromium.org
BUG=v8:5902

Review-Url: https://codereview.chromium.org/2786143004
Cr-Commit-Position: refs/heads/master@{#44316}
Committed:
https://chromium.googlesource.com/v8/v8/+/2de2840f2ec238226504fcf05bac6f01cfd...
==========

[commit-bot: I haz the power, 2017-03-31 18:27:40]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/2de2840f2ec238226504fcf05bac6f01cfd...