Changeset 195905 in webkit

Timestamp:

Jan 30, 2016 12:48:11 PM (8 years ago)

Author:

akling@apple.com

Message:

Shrink Heap::m_executables after cleaning it.
<​https://webkit.org/b/153682>

Reviewed by Darin Adler.

The Heap::m_executables Vector was never shrunk down, despite sometimes
getting pretty huge (~500kB in my longest-running WebContent process.)

After GC has finished pruning unmarked Executables, shrink the Vector.

• heap/Heap.cpp:

(JSC::Heap::clearUnmarkedExecutables):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• heap/Heap.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2016-01-30  Andreas Kling  <akling@apple.com>
+
+        Shrink Heap::m_executables after cleaning it.
+        <https://webkit.org/b/153682>
+
+        Reviewed by Darin Adler.
+
+        The Heap::m_executables Vector was never shrunk down, despite sometimes
+        getting pretty huge (~500kB in my longest-running WebContent process.)
+
+        After GC has finished pruning unmarked Executables, shrink the Vector.
+
+        * heap/Heap.cpp:
+        (JSC::Heap::clearUnmarkedExecutables):
+
 2016-01-29  Ada Chan  <adachan@apple.com>
 

trunk/Source/JavaScriptCore/heap/Heap.cpp

@@ -997 +997 @@
         m_executables.removeLast();
     }
+
+    m_executables.shrinkToFit();
 }