Changeset 231888 in webkit

Timestamp:

May 16, 2018 10:21:22 PM (6 years ago)

Author:

sbarati@apple.com

Message:

VM::shrinkFootprint should call collectNow(Sync) instead of collectSync so it also eagerly sweeps
​https://bugs.webkit.org/show_bug.cgi?id=185707

Reviewed by Mark Lam.

• runtime/VM.cpp:

(JSC::VM::shrinkFootprint):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• runtime/VM.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2018-05-16  Saam Barati  <sbarati@apple.com>
+
+        VM::shrinkFootprint should call collectNow(Sync) instead of collectSync so it also eagerly sweeps
+        https://bugs.webkit.org/show_bug.cgi?id=185707
+
+        Reviewed by Mark Lam.
+
+        * runtime/VM.cpp:
+        (JSC::VM::shrinkFootprint):
+
 2018-05-16  Caio Lima  <ticaiolima@gmail.com>
 

trunk/Source/JavaScriptCore/runtime/VM.cpp

@@ -779 +779 @@
     sanitizeStackForVM(this);
     deleteAllCode(DeleteAllCodeIfNotCollecting);
-    heap.collectSync();
+    heap.collectNow(Synchronousness::Sync);
     WTF::releaseFastMallocFreeMemory();
     // FIXME: Consider stopping various automatic threads here.