[stubs] Remove %_NewObject intrinsic

[stubs] Remove %_NewObject intrinsic

The last remaining JS user of this in promise.js has recently been moved
to TF. The underlying FastObjectStub is still in use.

BUG=

Review-Url: https://codereview.chromium.org/2598973002
Cr-Commit-Position: refs/heads/master@{#41919}
Committed: https://chromium.googlesource.com/v8/v8/+/132b7bf9e0b85c9b07970b1027617489ae719e87

[jgruber, 2016-12-22 08:29:15]

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

[commit-bot: I haz the power, 2016-12-22 08:29:25]

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-12-22 08:54:58]

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

[commit-bot: I haz the power, 2016-12-22 08:54:59]

Dry run: This issue passed the CQ dry run.

[jgruber, 2016-12-22 10:35:21]

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

[jgruber, 2016-12-22 10:35:22]

This removes access to %_NewObject from JS, but FastNewObjectStub.

[Benedikt Meurer, 2016-12-22 10:39:47]

Nice! LGTM.

[jgruber, 2016-12-22 10:52:52]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2016-12-22 10:53:04]

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-22 10:55:00]

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

[commit-bot: I haz the power, 2016-12-22 10:55:01]

Failed to apply patch for src/crankshaft/hydrogen.h:
While running git apply --index -p1;
  error: patch failed: src/crankshaft/hydrogen.h:2158
  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
1564702aa45c7a52f0ef72bb9458faa7a21f1ff5..0bad78822b80450cc8b908c036db35c3f5338a88
100644
--- a/src/crankshaft/hydrogen.h
+++ b/src/crankshaft/hydrogen.h
@@ -2158,7 +2158,6 @@ class HOptimizedGraphBuilder : public HGraphBuilder,
   F(IsTypedArray)                      \
   F(IsJSProxy)                         \
   F(Call)                              \
-  F(NewObject)                         \
   F(ToInteger)                         \
   F(ToObject)                          \
   F(ToString)                          \

[jgruber, 2016-12-22 12:05:09]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2016-12-22 12:05:21]

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-22 12:07:03]

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

[commit-bot: I haz the power, 2016-12-22 12:07:04]

Failed to apply patch for src/crankshaft/hydrogen.h:
While running git apply --index -p1;
  error: patch failed: src/crankshaft/hydrogen.h:2158
  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
1564702aa45c7a52f0ef72bb9458faa7a21f1ff5..0bad78822b80450cc8b908c036db35c3f5338a88
100644
--- a/src/crankshaft/hydrogen.h
+++ b/src/crankshaft/hydrogen.h
@@ -2158,7 +2158,6 @@ class HOptimizedGraphBuilder : public HGraphBuilder,
   F(IsTypedArray)                      \
   F(IsJSProxy)                         \
   F(Call)                              \
-  F(NewObject)                         \
   F(ToInteger)                         \
   F(ToObject)                          \
   F(ToString)                          \

[jgruber, 2016-12-22 12:07:37]

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

[commit-bot: I haz the power, 2016-12-22 12:07:41]

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-22 12:13:31]

The CQ bit was unchecked by jgruber@chromium.org

[jgruber, 2016-12-22 12:13:35]

The CQ bit was checked by jgruber@chromium.org

[jgruber, 2016-12-22 12:13:36]

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

[commit-bot: I haz the power, 2016-12-22 12:13:40]

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-22 12:37:19]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482408815993630,
"parent_rev": "8dad4feec9e45c1ba910808a388274f375584b5d", "commit_rev":
"132b7bf9e0b85c9b07970b1027617489ae719e87"}

[commit-bot: I haz the power, 2016-12-22 12:37:23]

Description was changed from

==========
[stubs] Remove %_NewObject intrinsic

The last remaining JS user of this in promise.js has recently been moved
to TF. The underlying FastObjectStub is still in use.

BUG=
==========

to

==========
[stubs] Remove %_NewObject intrinsic

The last remaining JS user of this in promise.js has recently been moved
to TF. The underlying FastObjectStub is still in use.

BUG=

Review-Url: https://codereview.chromium.org/2598973002
Cr-Commit-Position: refs/heads/master@{#41919}
Committed:
https://chromium.googlesource.com/v8/v8/+/132b7bf9e0b85c9b07970b1027617489ae7...
==========

[commit-bot: I haz the power, 2016-12-22 12:37:24]

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