Title: [231888] trunk/Source/_javascript_Core
Revision
231888
Author
sbar...@apple.com
Date
2018-05-16 22:21:22 -0700 (Wed, 16 May 2018)

Log 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):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (231887 => 231888)


--- trunk/Source/_javascript_Core/ChangeLog	2018-05-17 05:11:20 UTC (rev 231887)
+++ trunk/Source/_javascript_Core/ChangeLog	2018-05-17 05:21:22 UTC (rev 231888)
@@ -1,3 +1,13 @@
+2018-05-16  Saam Barati  <sbar...@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  <ticaiol...@gmail.com>
 
         [ESNext][BigInt] Implement support for "/" operation

Modified: trunk/Source/_javascript_Core/runtime/VM.cpp (231887 => 231888)


--- trunk/Source/_javascript_Core/runtime/VM.cpp	2018-05-17 05:11:20 UTC (rev 231887)
+++ trunk/Source/_javascript_Core/runtime/VM.cpp	2018-05-17 05:21:22 UTC (rev 231888)
@@ -778,7 +778,7 @@
 {
     sanitizeStackForVM(this);
     deleteAllCode(DeleteAllCodeIfNotCollecting);
-    heap.collectSync();
+    heap.collectNow(Synchronousness::Sync);
     WTF::releaseFastMallocFreeMemory();
     // FIXME: Consider stopping various automatic threads here.
     // https://bugs.webkit.org/show_bug.cgi?id=185447
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to