Fast body iteration for objects without unboxed fields.

Fast body iteration for objects without unboxed fields.

Currently BodyDescriptorBase::IterateBodyImpl does a run-time check
for obj->map()->HasFastPointerLayout(), which is redundant for all
non-JSObject heap objects.

BUG=chromium:694255
Review-Url: https://codereview.chromium.org/2774553003
Cr-Commit-Position: refs/heads/master@{#44111}
Committed: https://chromium.googlesource.com/v8/v8/+/dae6e43d38c0f66a49f66cfd7eb2f7113f054e68

[ulan, 2017-03-23 19:21:36]

ulan@chromium.org changed reviewers:
+ ishell@chromium.org

[ulan, 2017-03-23 19:21:36]

PTAL.

[ulan, 2017-03-24 14:07:32]

Description was changed from

==========
Add a variant of BodyDescriptorBase for objects with no unboxed fields.

Currently BodyDescriptorBase::IterateBodyImpl does a run-time check
for obj->map()->HasFastPointerLayout(), which is redundant for all
non-JSObject heap objects.

This patch introduces BodyDescriptorBase<PointerLayout::kFast> that
by-passes the run-time check.

BUG=chromium:694255
==========

to

==========
Fast body iteration for object without unboxed fields.

Currently BodyDescriptorBase::IterateBodyImpl does a run-time check
for obj->map()->HasFastPointerLayout(), which is redundant for all
non-JSObject heap objects.

BUG=chromium:694255
==========

[ulan, 2017-03-24 14:07:52]

Simplified as we discussed offline.

[Igor Sheludko, 2017-03-24 14:12:46]

A lot of unrelated formatting, but lgtm.

[ulan, 2017-03-24 14:13:38]

Description was changed from

==========
Fast body iteration for object without unboxed fields.

Currently BodyDescriptorBase::IterateBodyImpl does a run-time check
for obj->map()->HasFastPointerLayout(), which is redundant for all
non-JSObject heap objects.

BUG=chromium:694255
==========

to

==========
Fast body iteration for objects without unboxed fields.

Currently BodyDescriptorBase::IterateBodyImpl does a run-time check
for obj->map()->HasFastPointerLayout(), which is redundant for all
non-JSObject heap objects.

BUG=chromium:694255
==========

[ulan, 2017-03-24 14:14:02]

On 2017/03/24 14:12:46, Igor Sheludko wrote:
> A lot of unrelated formatting, but lgtm.

That's a result of git cl format :)

[ulan, 2017-03-24 14:14:06]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2017-03-24 14:14:15]

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-24 14:42:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490364846441590,
"parent_rev": "885f0bc4a77692e5aa3b2c0cad0c9f5b3ada3796", "commit_rev":
"dae6e43d38c0f66a49f66cfd7eb2f7113f054e68"}

[commit-bot: I haz the power, 2017-03-24 14:42:58]

Description was changed from

==========
Fast body iteration for objects without unboxed fields.

Currently BodyDescriptorBase::IterateBodyImpl does a run-time check
for obj->map()->HasFastPointerLayout(), which is redundant for all
non-JSObject heap objects.

BUG=chromium:694255
==========

to

==========
Fast body iteration for objects without unboxed fields.

Currently BodyDescriptorBase::IterateBodyImpl does a run-time check
for obj->map()->HasFastPointerLayout(), which is redundant for all
non-JSObject heap objects.

BUG=chromium:694255

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

[commit-bot: I haz the power, 2017-03-24 14:42:59]

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