Commit 0962f980 authored by sejunho's avatar sejunho Committed by Commit bot

Just clear semi-space mark bits once before Scavenge

Both PrepareForScavenge and ResetAllocationInfo clear semi-space mark bits before Scavenge. Doing this once is good enough.

BUG=

Review URL: https://codereview.chromium.org/1158623003

Cr-Commit-Position: refs/heads/master@{#28778}
parent b104a67e
...@@ -1595,8 +1595,6 @@ void Heap::Scavenge() { ...@@ -1595,8 +1595,6 @@ void Heap::Scavenge() {
SelectScavengingVisitorsTable(); SelectScavengingVisitorsTable();
incremental_marking()->PrepareForScavenge();
// Flip the semispaces. After flipping, to space is empty, from space has // Flip the semispaces. After flipping, to space is empty, from space has
// live objects. // live objects.
new_space_.Flip(); new_space_.Flip();
......
...@@ -578,16 +578,6 @@ void IncrementalMarking::MarkObjectGroups() { ...@@ -578,16 +578,6 @@ void IncrementalMarking::MarkObjectGroups() {
} }
void IncrementalMarking::PrepareForScavenge() {
if (!IsMarking()) return;
NewSpacePageIterator it(heap_->new_space()->FromSpaceStart(),
heap_->new_space()->FromSpaceEnd());
while (it.has_next()) {
Bitmap::Clear(it.next());
}
}
void IncrementalMarking::UpdateMarkingDequeAfterScavenge() { void IncrementalMarking::UpdateMarkingDequeAfterScavenge() {
if (!IsMarking()) return; if (!IsMarking()) return;
......
...@@ -73,8 +73,6 @@ class IncrementalMarking { ...@@ -73,8 +73,6 @@ class IncrementalMarking {
void MarkObjectGroups(); void MarkObjectGroups();
void PrepareForScavenge();
void UpdateMarkingDequeAfterScavenge(); void UpdateMarkingDequeAfterScavenge();
void Hurry(); void Hurry();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment