Mark maps having a hidden prototype rather than maps of hidden prototypes.

Mark maps having a hidden prototype rather than maps of hidden prototypes.

Generally we only care whether the next object is a hidden prototype.
It's simpler to check whether the current object has a hidden prototype
instead of walking to the next prototype and checking its map.

BUG=

Committed: https://crrev.com/d2503c4dbd50c666fe34536f7e7a3e8541640b1c
Cr-Commit-Position: refs/heads/master@{#33816}

[Toon Verwaest, 2016-02-08 11:58:21]

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

[commit-bot: I haz the power, 2016-02-08 11:58:27]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1675223002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1675223002/1

[Toon Verwaest, 2016-02-08 12:05:03]

verwaest@chromium.org changed reviewers:
+ cbruni@chromium.org

[Toon Verwaest, 2016-02-08 12:05:03]

ptal

[Yang, 2016-02-08 12:23:27]

On 2016/02/08 12:05:03, Toon Verwaest wrote:
> ptal

debug lgtm.

[commit-bot: I haz the power, 2016-02-08 12:28:48]

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

[commit-bot: I haz the power, 2016-02-08 12:28:49]

Dry run: This issue passed the CQ dry run.

[Camillo Bruni, 2016-02-08 13:24:09]

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

[Camillo Bruni, 2016-02-08 13:24:09]

lgtm with one comment on x87

https://codereview.chromium.org/1675223002/diff/1/src/builtins.cc
File src/builtins.cc (right):

https://codereview.chromium.org/1675223002/diff/1/src/builtins.cc#newcode2082
src/builtins.cc:2082: Handle<JSReceiver> receiver =
Handle<JSReceiver>::cast(target);
Probably 20% of v8's code-base consists of casts by now :D

https://codereview.chromium.org/1675223002/diff/1/src/x87/builtins-x87.cc
File src/x87/builtins-x87.cc (right):

https://codereview.chromium.org/1675223002/diff/1/src/x87/builtins-x87.cc#new...
src/x87/builtins-x87.cc:2693: Immediate(Map::IsHiddenPrototype::kMask));
missed ::HasHiddenPrototype

[Toon Verwaest, 2016-02-08 13:28:39]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2016-02-08 13:28:39]

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

[commit-bot: I haz the power, 2016-02-08 13:28:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1675223002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1675223002/20001

[commit-bot: I haz the power, 2016-02-08 13:49:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-08 13:50:29]

Description was changed from

==========
Mark maps having a hidden prototype rather than maps of hidden prototypes.

Generally we only care whether the next object is a hidden prototype.
It's simpler to check whether the current object has a hidden prototype
instead of walking to the next prototype and checking its map.

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

to

==========
Mark maps having a hidden prototype rather than maps of hidden prototypes.

Generally we only care whether the next object is a hidden prototype.
It's simpler to check whether the current object has a hidden prototype
instead of walking to the next prototype and checking its map.

BUG=

Committed: https://crrev.com/d2503c4dbd50c666fe34536f7e7a3e8541640b1c
Cr-Commit-Position: refs/heads/master@{#33816}
==========

[commit-bot: I haz the power, 2016-02-08 13:50:30]

Patchset 2 (id:??) landed as
https://crrev.com/d2503c4dbd50c666fe34536f7e7a3e8541640b1c
Cr-Commit-Position: refs/heads/master@{#33816}