Reland of [heap] Refactor evacuation verifier

Reland of [heap] Refactor evacuation verifier

This reverts commit c766727ae69d43db66415a3483acc0e846b444d7.

BUG=chromium:651354
Review-Url: https://codereview.chromium.org/2793323002
Cr-Commit-Position: refs/heads/master@{#44386}
Committed: https://chromium.googlesource.com/v8/v8/+/6d89de7b9e5fc3a5f9d0fa2cef51864191d5ff5f

[Michael Lippautz, 2017-04-04 14:48:54]

Description was changed from

==========
Reland of [heap] Refactor evacuation verifier

This reverts commit c766727ae69d43db66415a3483acc0e846b444d7.

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

to

==========
Reland of [heap] Refactor evacuation verifier

This reverts commit c766727ae69d43db66415a3483acc0e846b444d7.

BUG=chromium:651354
==========

[Michael Lippautz, 2017-04-04 14:50:18]

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

[Michael Lippautz, 2017-04-04 14:50:27]

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

[Michael Lippautz, 2017-04-04 14:50:27]

ptal

https://codereview.chromium.org/2793323002/diff/20001/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/2793323002/diff/20001/src/heap/mark-compact.c...
src/heap/mark-compact.cc:262: if (p->Contains(space->top()))
Previously we used the HeapObjectIterator for this one. However, all other
marking and evacuation methods just iterate using raw addresses and size.
Changing this one requires us to write a filler in the linear allocation area of
old space.

[commit-bot: I haz the power, 2017-04-04 14:50:30]

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

[ulan, 2017-04-04 14:52:09]

lgtm

[commit-bot: I haz the power, 2017-04-04 15:18:05]

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

[commit-bot: I haz the power, 2017-04-04 15:18:05]

Dry run: This issue passed the CQ dry run.

[Michael Lippautz, 2017-04-04 16:09:52]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2017-04-04 16:10: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, 2017-04-04 16:11:57]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491322192921390,
"parent_rev": "1769c7034ba4203c6301d5db5e3081b47585af0d", "commit_rev":
"6d89de7b9e5fc3a5f9d0fa2cef51864191d5ff5f"}

[commit-bot: I haz the power, 2017-04-04 16:12:03]

Description was changed from

==========
Reland of [heap] Refactor evacuation verifier

This reverts commit c766727ae69d43db66415a3483acc0e846b444d7.

BUG=chromium:651354
==========

to

==========
Reland of [heap] Refactor evacuation verifier

This reverts commit c766727ae69d43db66415a3483acc0e846b444d7.

BUG=chromium:651354

Review-Url: https://codereview.chromium.org/2793323002
Cr-Commit-Position: refs/heads/master@{#44386}
Committed:
https://chromium.googlesource.com/v8/v8/+/6d89de7b9e5fc3a5f9d0fa2cef51864191d...
==========

[commit-bot: I haz the power, 2017-04-04 16:12:04]

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