[heap] Reland "Remove size specializations in static object visitors. (patchset #4 id:60001 of http…

[heap] Reland "Remove size specializations in static object visitors. (patchset #4 id:60001 of https://codereview.chromium.org/2763413007/ )"

This reverts commit 61df2d90a952066cec6eb20fc0602b5fae749874.
The CL was speculatively reverted due to canary crashes, which turned
out to be caused by another CL.

Original issue's description:
> [heap] Remove size specializations in static object visitors.
>
> Apart from that this patch adds kVisitJSObjectFast for JSObjects that
> do not have any unboxed double fields and can be visited without
> run-time layout check.
>
> BUG=chromium:694255
>
> Review-Url: https://codereview.chromium.org/2763413007
> Cr-Commit-Position: refs/heads/master@{#44237}
> Committed: https://chromium.googlesource.com/v8/v8/+/dbb1cbe3a85d5c5528ce876d905e78d2ab35f00b

Review-Url: https://codereview.chromium.org/2808533002
Cr-Commit-Position: refs/heads/master@{#44494}
Committed: https://chromium.googlesource.com/v8/v8/+/e86d3a2ac65c336c3c611336de1ea4773495b5cf

[ulan, 2017-04-07 19:15:38]

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

[ulan, 2017-04-07 19:15:38]

ptal

[ulan, 2017-04-07 19:16:50]

Description was changed from

==========
[heap] Reland "Remove size specializations in static object visitors. (patchset
#4 id:60001 of https://codereview.chromium.org/2763413007/ )"

This reverts commit 61df2d90a952066cec6eb20fc0602b5fae749874.

Original issue's description:
> [heap] Remove size specializations in static object visitors.
>
> Apart from that this patch adds kVisitJSObjectFast for JSObjects that
> do not have any unboxed double fields and can be visited without
> run-time layout check.
>
> BUG=chromium:694255
>
> Review-Url: https://codereview.chromium.org/2763413007
> Cr-Commit-Position: refs/heads/master@{#44237}
> Committed:
https://chromium.googlesource.com/v8/v8/+/dbb1cbe3a85d5c5528ce876d905e78d2ab3...
==========

to

==========
[heap] Reland "Remove size specializations in static object visitors. (patchset
#4 id:60001 of https://codereview.chromium.org/2763413007/ )"

This reverts commit 61df2d90a952066cec6eb20fc0602b5fae749874.
The CL was speculatively reverted due to canary crashes, which turned
out to be caused by another CL.

Original issue's description:
> [heap] Remove size specializations in static object visitors.
>
> Apart from that this patch adds kVisitJSObjectFast for JSObjects that
> do not have any unboxed double fields and can be visited without
> run-time layout check.
>
> BUG=chromium:694255
>
> Review-Url: https://codereview.chromium.org/2763413007
> Cr-Commit-Position: refs/heads/master@{#44237}
> Committed:
https://chromium.googlesource.com/v8/v8/+/dbb1cbe3a85d5c5528ce876d905e78d2ab3...
==========

[Michael Lippautz, 2017-04-07 19:17:45]

lgtm

[ulan, 2017-04-07 19:19:25]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2017-04-07 19:19:35]

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-04-07 19:53:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491592765965530, "parent_rev":
"749c00ef454d08807d659ebcbfb92169e2d1dd06", "commit_rev":
"e86d3a2ac65c336c3c611336de1ea4773495b5cf"}

[commit-bot: I haz the power, 2017-04-07 19:54:02]

Description was changed from

==========
[heap] Reland "Remove size specializations in static object visitors. (patchset
#4 id:60001 of https://codereview.chromium.org/2763413007/ )"

This reverts commit 61df2d90a952066cec6eb20fc0602b5fae749874.
The CL was speculatively reverted due to canary crashes, which turned
out to be caused by another CL.

Original issue's description:
> [heap] Remove size specializations in static object visitors.
>
> Apart from that this patch adds kVisitJSObjectFast for JSObjects that
> do not have any unboxed double fields and can be visited without
> run-time layout check.
>
> BUG=chromium:694255
>
> Review-Url: https://codereview.chromium.org/2763413007
> Cr-Commit-Position: refs/heads/master@{#44237}
> Committed:
https://chromium.googlesource.com/v8/v8/+/dbb1cbe3a85d5c5528ce876d905e78d2ab3...
==========

to

==========
[heap] Reland "Remove size specializations in static object visitors. (patchset
#4 id:60001 of https://codereview.chromium.org/2763413007/ )"

This reverts commit 61df2d90a952066cec6eb20fc0602b5fae749874.
The CL was speculatively reverted due to canary crashes, which turned
out to be caused by another CL.

Original issue's description:
> [heap] Remove size specializations in static object visitors.
>
> Apart from that this patch adds kVisitJSObjectFast for JSObjects that
> do not have any unboxed double fields and can be visited without
> run-time layout check.
>
> BUG=chromium:694255
>
> Review-Url: https://codereview.chromium.org/2763413007
> Cr-Commit-Position: refs/heads/master@{#44237}
> Committed:
https://chromium.googlesource.com/v8/v8/+/dbb1cbe3a85d5c5528ce876d905e78d2ab3...

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

[commit-bot: I haz the power, 2017-04-07 19:54:03]

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