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

Timestamp:

Jan 21, 2015, 6:56:26 PM (10 years ago)

Author:

[email protected]

Message:

Change Heap::m_compiledCode to use a Vector
​https://bugs.webkit.org/show_bug.cgi?id=140717

Reviewed by Andreas Kling.

Right now it's a DoublyLinkedList, which is iterated during each
collection. This contributes to some of the longish Eden pause times.
A Vector would be more appropriate and would also allow ExecutableBase
to be 2 pointers smaller.

• heap/Heap.cpp:

(JSC::Heap::deleteAllCompiledCode):
(JSC::Heap::deleteAllUnlinkedFunctionCode):
(JSC::Heap::clearUnmarkedExecutables):

• heap/Heap.h:
• runtime/Executable.h: No longer need to inherit from DoublyLinkedListNode.

File:

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

trunk/Source/JavaScriptCore/heap/Heap.h

@@ -377 +377 @@
     double m_lastCodeDiscardTime;
 
-    DoublyLinkedList<ExecutableBase> m_compiledCode;
+    Vector<ExecutableBase*> m_compiledCode;
     
     RefPtr<GCActivityCallback> m_fullActivityCallback;