Managed MarkedBlocks in a linked list instead of a vector, so arbitrary removal is O(1)
Created attachment 83389 [details] Patch
Comment on attachment 83389 [details] Patch r=me
Committed r79367: <http://trac.webkit.org/changeset/79367>
This patch caused all tests to crash on Snow Leopard: http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r79366%20(25764)/ http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r79367%20(25765)/results.html I'm rolling this patch per olliej's suggestion.
Reopening, since it was rolled out.
Created attachment 83513 [details] Patch
Comment on attachment 83513 [details] Patch This patch does not include the DoublyLinkedList.h header, but includes many references to it!
Created attachment 83516 [details] Patch
Comment on attachment 83516 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=83516&action=review > Source/JavaScriptCore/runtime/MarkedSpace.cpp:58 > + MarkedBlock* next = 0; Doesn’t seem helpful to initialize this here.
Committed r79472: <http://trac.webkit.org/changeset/79472>