[regexp] Remove IsRegExp intrinsic

[regexp] Remove IsRegExp intrinsic

The two remaining uses of this intrinsic in debug.js and mirrors.js now
simply rely on the runtime function.

BUG=v8:5339
Review-Url: https://codereview.chromium.org/2591923003
Cr-Original-Commit-Position: refs/heads/master@{#41892}
Committed: https://chromium.googlesource.com/v8/v8/+/c9cb94a06fa7a863d24dd6760b66cecd55748abf
Review-Url: https://codereview.chromium.org/2591923003
Cr-Commit-Position: refs/heads/master@{#42128}
Committed: https://chromium.googlesource.com/v8/v8/+/9e65ecd957e62a1477d21742c61014fd0421521f

[jgruber, 2016-12-21 13:24:04]

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

[commit-bot: I haz the power, 2016-12-21 13:24:07]

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

[jgruber, 2016-12-21 13:46:55]

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

[Yang, 2016-12-21 13:49:29]

On 2016/12/21 13:46:58, jgruber wrote:

awesome. lgtm.

[commit-bot: I haz the power, 2016-12-21 13:49:57]

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

[commit-bot: I haz the power, 2016-12-21 13:49:58]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-12-21 13:50:47]

W00t. LGTM!

[jgruber, 2016-12-21 13:53:46]

The CQ bit was checked by jgruber@chromium.org

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

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-12-21 13:55:28]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482328426876130, "parent_rev":
"55fc5c0c327cec8385503bdcda0cfcee7f51213e", "commit_rev":
"c9cb94a06fa7a863d24dd6760b66cecd55748abf"}

[commit-bot: I haz the power, 2016-12-21 13:55:32]

Description was changed from

==========
[regexp] Remove IsRegExp intrinsic

The two remaining uses of this intrinsic in debug.js and mirrors.js now
simply rely on the runtime function.

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

to

==========
[regexp] Remove IsRegExp intrinsic

The two remaining uses of this intrinsic in debug.js and mirrors.js now
simply rely on the runtime function.

BUG=v8:5339

Review-Url: https://codereview.chromium.org/2591923003
Cr-Commit-Position: refs/heads/master@{#41892}
Committed:
https://chromium.googlesource.com/v8/v8/+/c9cb94a06fa7a863d24dd6760b66cecd557...
==========

[commit-bot: I haz the power, 2016-12-21 13:55:33]

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

[Michael Hablich, 2016-12-22 09:39:15]

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

The reason for reverting is: speculative revert:
https://codereview.chromium.org/2596013002/.

[jgruber, 2016-12-22 11:51:46]

Description was changed from

==========
[regexp] Remove IsRegExp intrinsic

The two remaining uses of this intrinsic in debug.js and mirrors.js now
simply rely on the runtime function.

BUG=v8:5339

Review-Url: https://codereview.chromium.org/2591923003
Cr-Commit-Position: refs/heads/master@{#41892}
Committed:
https://chromium.googlesource.com/v8/v8/+/c9cb94a06fa7a863d24dd6760b66cecd557...
==========

to

==========
[regexp] Remove IsRegExp intrinsic

The two remaining uses of this intrinsic in debug.js and mirrors.js now
simply rely on the runtime function.

BUG=v8:5339

Review-Url: https://codereview.chromium.org/2591923003
Cr-Commit-Position: refs/heads/master@{#41892}
Committed:
https://chromium.googlesource.com/v8/v8/+/c9cb94a06fa7a863d24dd6760b66cecd557...
==========

[jgruber, 2017-01-09 08:14:54]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2017-01-09 08:14:59]

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-01-09 08:41:03]

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

[commit-bot: I haz the power, 2017-01-09 08:41:05]

Failed to apply patch for src/crankshaft/hydrogen.h:
While running git apply --index -p1;
  error: patch failed: src/crankshaft/hydrogen.h:2156
  error: src/crankshaft/hydrogen.h: patch does not apply

Patch:       src/crankshaft/hydrogen.h
Index: src/crankshaft/hydrogen.h
diff --git a/src/crankshaft/hydrogen.h b/src/crankshaft/hydrogen.h
index
60a6f743a28a5a06b0959c17880b70aae737b598..1564702aa45c7a52f0ef72bb9458faa7a21f1ff5
100644
--- a/src/crankshaft/hydrogen.h
+++ b/src/crankshaft/hydrogen.h
@@ -2156,7 +2156,6 @@ class HOptimizedGraphBuilder : public HGraphBuilder,
   F(IsSmi)                             \
   F(IsArray)                           \
   F(IsTypedArray)                      \
-  F(IsRegExp)                          \
   F(IsJSProxy)                         \
   F(Call)                              \
   F(NewObject)                         \

[jgruber, 2017-01-09 08:44:20]

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

[commit-bot: I haz the power, 2017-01-09 08:44:29]

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

[jgruber, 2017-01-09 08:45:02]

On 2016/12/22 09:39:15, Hablich (OOO) wrote:
> A revert of this CL (patchset #1 id:1) has been created in
> https://codereview.chromium.org/2592383002/ by mailto:hablich@chromium.org.
> 
> The reason for reverting is: speculative revert:
> https://codereview.chromium.org/2596013002/.

Relanding this, seems unrelated to that failure.

[jgruber, 2017-01-09 08:55:52]

The CQ bit was unchecked by jgruber@chromium.org

[jgruber, 2017-01-09 08:55:58]

The CQ bit was checked by jgruber@chromium.org

[jgruber, 2017-01-09 08:55:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
bmeurer@chromium.org, yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2591923003/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2017-01-09 08:56:05]

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

[commit-bot: I haz the power, 2017-01-09 08:56:06]

This CL has an open dependency (Issue 2619753002 Patch 1). Please resolve the
dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[jgruber, 2017-01-09 09:32:55]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2017-01-09 09:33:07]

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-01-09 09:34:49]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483954375222240,
"parent_rev": "1c26e0a4934b714be8f63fe0afcd545f12f26645", "commit_rev":
"9e65ecd957e62a1477d21742c61014fd0421521f"}

[commit-bot: I haz the power, 2017-01-09 09:34:55]

Description was changed from

==========
[regexp] Remove IsRegExp intrinsic

The two remaining uses of this intrinsic in debug.js and mirrors.js now
simply rely on the runtime function.

BUG=v8:5339

Review-Url: https://codereview.chromium.org/2591923003
Cr-Commit-Position: refs/heads/master@{#41892}
Committed:
https://chromium.googlesource.com/v8/v8/+/c9cb94a06fa7a863d24dd6760b66cecd557...
==========

to

==========
[regexp] Remove IsRegExp intrinsic

The two remaining uses of this intrinsic in debug.js and mirrors.js now
simply rely on the runtime function.

BUG=v8:5339

Review-Url: https://codereview.chromium.org/2591923003
Cr-Original-Commit-Position: refs/heads/master@{#41892}
Committed:
https://chromium.googlesource.com/v8/v8/+/c9cb94a06fa7a863d24dd6760b66cecd557...
Review-Url: https://codereview.chromium.org/2591923003
Cr-Commit-Position: refs/heads/master@{#42128}
Committed:
https://chromium.googlesource.com/v8/v8/+/9e65ecd957e62a1477d21742c61014fd042...
==========

[commit-bot: I haz the power, 2017-01-09 09:34:57]

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