Changeset 92251 in webkit for trunk/Source/JavaScriptCore/heap/NewSpace.cpp

Timestamp:

Aug 2, 2011, 6:48:13 PM (14 years ago)

Author:

[email protected]

Message:

JSC GC may not be able to reuse partially-free blocks after a
full collection
​https://bugs.webkit.org/show_bug.cgi?id=65585

Reviewed by Darin Adler.

This fixes the linked list management bug. This fix is performance
neutral on SunSpider.

• heap/NewSpace.cpp:

(JSC::NewSpace::removeBlock):

File:

• trunk/Source/JavaScriptCore/heap/NewSpace.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/heap/NewSpace.cpp

@@ -59 +59 @@
     block->setInNewSpace(false);
     SizeClass& sizeClass = sizeClassFor(block->cellSize());
-    sizeClass.nextBlock = block->next();
+    if (sizeClass.nextBlock == block)
+        sizeClass.nextBlock = block->next();
     sizeClass.blockList.remove(block);
 }