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

Timestamp:

May 23, 2012, 4:55:27 PM (13 years ago)

Author:

[email protected]

Message:

Refactored WeakBlock to use malloc, clarify behavior
​https://bugs.webkit.org/show_bug.cgi?id=87318

Reviewed by Filip Pizlo.

We want to use malloc so we can make these smaller than 4KB,
since an individual MarkedBlock will usually have fewer than
4KB worth of weak pointers.

• heap/Heap.cpp:

(JSC::Heap::markRoots): Renamed visitLiveWeakImpls to visit, since
we no longer need to distinguish from "visitDeadWeakImpls".

Renamed "visitDeadWeakImpls" to "reap" because we're not actually
doing any visiting -- we're just tagging things as dead.

• heap/WeakBlock.cpp:

(JSC::WeakBlock::create):
(JSC::WeakBlock::destroy):
(JSC::WeakBlock::WeakBlock): Malloc!

(JSC::WeakBlock::visit):
(JSC::WeakBlock::reap): Renamed as above.

• heap/WeakBlock.h:

(WeakBlock): Reduced to 3KB, as explained above.

• heap/WeakSet.cpp:

(JSC::WeakSet::visit):
(JSC::WeakSet::reap):

• heap/WeakSet.h:

(WeakSet): Updated for renames, and to match WebKit style.

File:

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

trunk/Source/JavaScriptCore/heap/Heap.cpp

@@ -562 +562 @@
         GCPHASE(VisitingLiveWeakHandles);
         while (true) {
-            m_weakSet.visitLiveWeakImpls(heapRootVisitor);
+            m_weakSet.visit(heapRootVisitor);
             harvestWeakReferences();
             if (visitor.isEmpty())
@@ -577 +577 @@
 
     {
-        GCPHASE(VisitingDeadWeakHandles);
-        m_weakSet.visitDeadWeakImpls(heapRootVisitor);
+        GCPHASE(ReapingWeakHandles);
+        m_weakSet.reap();
     }