Changeset 189524 in webkit for trunk/Source/JavaScriptCore/heap/Heap.cpp

Timestamp:

Sep 8, 2015, 6:28:22 PM (10 years ago)

Author:

[email protected]

Message:

Unreviewed, rolling out r189516.
​https://bugs.webkit.org/show_bug.cgi?id=148989

broke tests on windows (Requested by alexchristensen on
#webkit).

Reverted changeset:

"GC should be able to discover new strong CodeBlock references
during marking"
​https://bugs.webkit.org/show_bug.cgi?id=148981
​http://trac.webkit.org/changeset/189516

File:

• trunk/Source/JavaScriptCore/heap/Heap.cpp (modified) (3 diffs)

trunk/Source/JavaScriptCore/heap/Heap.cpp

@@ -526 +526 @@
 
 #if ENABLE(DFG_JIT)
-    DFG::clearCodeBlockMarks(*m_vm);
+    DFG::clearCodeBlockMarks(*m_vm, m_codeBlocks);
 #endif
     if (m_operationInProgress == EdenCollection)
@@ -662 +662 @@
 #if ENABLE(DFG_JIT)
     for (auto worklist : m_suspendedCompilerWorklists)
-        worklist->visitWeakReferences(m_slotVisitor);
+        worklist->visitWeakReferences(m_slotVisitor, m_codeBlocks);
 
     if (Options::logGC() == GCLogging::Verbose)
@@ -769 +769 @@
         harvestWeakReferences();
         visitCompilerWorklistWeakReferences();
+        m_codeBlocks.traceMarked(m_slotVisitor); // New "executing" code blocks may be discovered.
         if (m_slotVisitor.isEmpty())
             break;