Changeset 115092 in webkit for trunk/Source/JavaScriptCore/heap/Heap.h

Timestamp:

Apr 24, 2012, 12:11:12 PM (13 years ago)

Author:

[email protected]

Message:

"GlobalHandle" HandleHeap (now WeakSet) allocations grow but do not shrink
​https://bugs.webkit.org/show_bug.cgi?id=84740
<rdar://problem/9917638>

Reviewed by Gavin Barraclough.

Shrink!

• heap/Heap.cpp:

(JSC::Heap::destroy): Be more specific about what's shrinking, since we
can also shrink the WeakSet, but we don't do so here.

(JSC::Heap::collect): If we're going to shrink the heap, shrink the
WeakSet too. Otherwise, its footprint is permanent.

• heap/Heap.h:

(Heap): Removed shrink() as a public interface, since it's vague about
which parts of the heap it affects, and it's really an internal detail.

• heap/WeakSet.cpp:

(JSC::WeakSet::shrink): Nix any free blocks. We assume that sweep() has
already taken place, since that's the convention for shrink() in the heap.

• heap/WeakSet.h:

(WeakSet): New function!

File:

• trunk/Source/JavaScriptCore/heap/Heap.h (modified) (1 diff)

trunk/Source/JavaScriptCore/heap/Heap.h

@@ -192 +192 @@
         void finalizeUnconditionalFinalizers();
         
-        void shrink();
         void releaseFreeBlocks();
         void sweep();