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

Side by Side Diff: src/heap/mark-compact.cc

Issue 2604583002: [heap] Report wrappers after processing the marking deque incrementally (Closed)
Patch Set: Fix test Created 3 years, 12 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 unified diff | Download patch
« no previous file with comments | « src/heap/incremental-marking.cc ('k') | test/unittests/heap/embedder-tracing-unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/heap/mark-compact.h" 5 #include "src/heap/mark-compact.h"
6 6
7 #include "src/base/atomicops.h" 7 #include "src/base/atomicops.h"
8 #include "src/base/bits.h" 8 #include "src/base/bits.h"
9 #include "src/base/sys-info.h" 9 #include "src/base/sys-info.h"
10 #include "src/code-stubs.h" 10 #include "src/code-stubs.h"
(...skipping 2118 matching lines...) Expand 10 before | Expand all | Expand 10 after
2129 // Mark all objects reachable (transitively) from objects on the marking 2129 // Mark all objects reachable (transitively) from objects on the marking
2130 // stack including references only considered in the atomic marking pause. 2130 // stack including references only considered in the atomic marking pause.
2131 void MarkCompactCollector::ProcessEphemeralMarking( 2131 void MarkCompactCollector::ProcessEphemeralMarking(
2132 ObjectVisitor* visitor, bool only_process_harmony_weak_collections) { 2132 ObjectVisitor* visitor, bool only_process_harmony_weak_collections) {
2133 DCHECK(marking_deque()->IsEmpty() && !marking_deque()->overflowed()); 2133 DCHECK(marking_deque()->IsEmpty() && !marking_deque()->overflowed());
2134 bool work_to_do = true; 2134 bool work_to_do = true;
2135 while (work_to_do) { 2135 while (work_to_do) {
2136 if (!only_process_harmony_weak_collections) { 2136 if (!only_process_harmony_weak_collections) {
2137 if (heap_->local_embedder_heap_tracer()->InUse()) { 2137 if (heap_->local_embedder_heap_tracer()->InUse()) {
2138 TRACE_GC(heap()->tracer(), GCTracer::Scope::MC_MARK_WRAPPER_TRACING); 2138 TRACE_GC(heap()->tracer(), GCTracer::Scope::MC_MARK_WRAPPER_TRACING);
2139 heap_->local_embedder_heap_tracer()->RegisterWrappersWithRemoteTracer();
2139 heap_->local_embedder_heap_tracer()->Trace( 2140 heap_->local_embedder_heap_tracer()->Trace(
2140 0, 2141 0,
2141 EmbedderHeapTracer::AdvanceTracingActions( 2142 EmbedderHeapTracer::AdvanceTracingActions(
2142 EmbedderHeapTracer::ForceCompletionAction::FORCE_COMPLETION)); 2143 EmbedderHeapTracer::ForceCompletionAction::FORCE_COMPLETION));
2143 } else { 2144 } else {
2144 TRACE_GC(heap()->tracer(), GCTracer::Scope::MC_MARK_OBJECT_GROUPING); 2145 TRACE_GC(heap()->tracer(), GCTracer::Scope::MC_MARK_OBJECT_GROUPING);
2145 isolate()->global_handles()->IterateObjectGroups( 2146 isolate()->global_handles()->IterateObjectGroups(
2146 visitor, &IsUnmarkedHeapObjectWithHeap); 2147 visitor, &IsUnmarkedHeapObjectWithHeap);
2147 MarkImplicitRefGroups(&MarkCompactMarkingVisitor::MarkObject); 2148 MarkImplicitRefGroups(&MarkCompactMarkingVisitor::MarkObject);
2148 } 2149 }
(...skipping 1932 matching lines...) Expand 10 before | Expand all | Expand 10 after
4081 // The target is always in old space, we don't have to record the slot in 4082 // The target is always in old space, we don't have to record the slot in
4082 // the old-to-new remembered set. 4083 // the old-to-new remembered set.
4083 DCHECK(!heap()->InNewSpace(target)); 4084 DCHECK(!heap()->InNewSpace(target));
4084 RecordRelocSlot(host, &rinfo, target); 4085 RecordRelocSlot(host, &rinfo, target);
4085 } 4086 }
4086 } 4087 }
4087 } 4088 }
4088 4089
4089 } // namespace internal 4090 } // namespace internal
4090 } // namespace v8 4091 } // namespace v8
OLDNEW
« no previous file with comments | « src/heap/incremental-marking.cc ('k') | test/unittests/heap/embedder-tracing-unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698