Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(638)

Unified Diff: src/heap/spaces.cc

Issue 2364603002: Revert of [heap] Reland Concurrently free empty typed slot set chunks. (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/heap/spaces.h ('k') | test/unittests/heap/slot-set-unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/heap/spaces.cc
diff --git a/src/heap/spaces.cc b/src/heap/spaces.cc
index 1149dc51c367803c904b6c6ed5b0c0ba5dd6f401..8982a2953273abbe35993b57ab63ee43d42ae0d7 100644
--- a/src/heap/spaces.cc
+++ b/src/heap/spaces.cc
@@ -510,7 +510,7 @@
chunk->InitializeReservedMemory();
chunk->old_to_new_slots_ = nullptr;
chunk->old_to_old_slots_ = nullptr;
- chunk->typed_old_to_new_slots_.SetValue(nullptr);
+ chunk->typed_old_to_new_slots_ = nullptr;
chunk->typed_old_to_old_slots_ = nullptr;
chunk->skip_list_ = nullptr;
chunk->write_barrier_counter_ = kWriteBarrierCounterGranularity;
@@ -1077,7 +1077,7 @@
}
if (old_to_new_slots_ != nullptr) ReleaseOldToNewSlots();
if (old_to_old_slots_ != nullptr) ReleaseOldToOldSlots();
- if (typed_old_to_new_slots_.Value() != nullptr) ReleaseTypedOldToNewSlots();
+ if (typed_old_to_new_slots_ != nullptr) ReleaseTypedOldToNewSlots();
if (typed_old_to_old_slots_ != nullptr) ReleaseTypedOldToOldSlots();
if (local_tracker_ != nullptr) ReleaseLocalTracker();
}
@@ -1113,14 +1113,13 @@
}
void MemoryChunk::AllocateTypedOldToNewSlots() {
- DCHECK(nullptr == typed_old_to_new_slots_.Value());
- typed_old_to_new_slots_.SetValue(new TypedSlotSet(address()));
+ DCHECK(nullptr == typed_old_to_new_slots_);
+ typed_old_to_new_slots_ = new TypedSlotSet(address());
}
void MemoryChunk::ReleaseTypedOldToNewSlots() {
- TypedSlotSet* typed_old_to_new_slots = typed_old_to_new_slots_.Value();
- delete typed_old_to_new_slots;
- typed_old_to_new_slots_.SetValue(nullptr);
+ delete typed_old_to_new_slots_;
+ typed_old_to_new_slots_ = nullptr;
}
void MemoryChunk::AllocateTypedOldToOldSlots() {
« no previous file with comments | « src/heap/spaces.h ('k') | test/unittests/heap/slot-set-unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698