Skip to content
forked from v8/v8

Commit

Permalink
[heap] Fix aborting compaction with map space compaction
Browse files Browse the repository at this point in the history
When aborting compaction on a page, slots need to be recorded in the
non-evacuated part. Now that maps might be recorded as well, slots
need to be recorded in the map word as well.

Bug: chromium:1359294, v8:12578
Change-Id: I91e4a98ae2951d814d4b50e527b34d8e54d55434
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3871297
Commit-Queue: Omer Katz <[email protected]>
Reviewed-by: Omer Katz <[email protected]>
Auto-Submit: Dominik Inführ <[email protected]>
Cr-Commit-Position: refs/heads/main@{#82951}
  • Loading branch information
Dominik Inführ authored and V8 LUCI CQ committed Sep 2, 2022
1 parent bbc3883 commit 3ec02e3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/heap/mark-compact.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2040,7 +2040,7 @@ class EvacuateRecordOnlyVisitor final : public HeapObjectVisitor {
// Instead of calling object.IterateBodyFast(cage_base(), &visitor) here
// we can shortcut and use the precomputed size value passed to the visitor.
DCHECK_EQ(object.SizeFromMap(map), size);
object.IterateBodyFast(map, size, &visitor);
object.IterateFast(map, size, &visitor);
return true;
}

Expand Down

0 comments on commit 3ec02e3

Please sign in to comment.