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

Unified Diff: runtime/vm/object.cc

Issue 2995543004: [vm, gc] Require a safepoint for heap iteration. (Closed)
Patch Set: . Created 3 years, 4 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
Index: runtime/vm/object.cc
diff --git a/runtime/vm/object.cc b/runtime/vm/object.cc
index d1b188a1065a65122be9dafa4072899cb07673fc..fd0c7afa79d614ee2ac79ad125005573b700f7b4 100644
--- a/runtime/vm/object.cc
+++ b/runtime/vm/object.cc
@@ -1117,9 +1117,10 @@ void Object::FinalizeVMIsolate(Isolate* isolate) {
{
ASSERT(isolate == Dart::vm_isolate());
WritableVMIsolateScope scope(Thread::Current());
+ HeapIterationScope iteration;
FinalizeVMIsolateVisitor premarker;
ASSERT(isolate->heap()->UsedInWords(Heap::kNew) == 0);
- isolate->heap()->IterateOldObjectsNoImagePages(&premarker);
+ iteration.IterateOldObjectsNoImagePages(&premarker);
// Make the VM isolate read-only again after setting all objects as marked.
// Note objects in image pages are already pre-marked.
}
@@ -14126,7 +14127,7 @@ RawCode* Code::LookupCodeInIsolate(Isolate* isolate, uword pc) {
if (isolate->heap() == NULL) {
return Code::null();
}
- NoSafepointScope no_safepoint;
+ HeapIterationScope heap_iteration_scope;
SlowFindRawCodeVisitor visitor(pc);
RawObject* needle = isolate->heap()->FindOldObject(&visitor);
if (needle != Code::null()) {

Powered by Google App Engine
This is Rietveld 408576698