Currently, Heap::allocate is searching for a new cell in the bitmap a single bit at a time. This can be improved by searching a word at a time in the bitmap when our first guess fails.
Created attachment 57126 [details] suggested patch
Comment on attachment 57126 [details] suggested patch + if (!~block->marked.bits[m_heap.nextCell >> 5]) + m_heap.nextCell = (m_heap.nextCell & (~0x1F)) + 32; I think this code might be wrong on 64bit. Also, would be nice to encapsulate these bitops in the CollectorBitmap interface. Let's talk about this in person.
Created attachment 57132 [details] Moved the advance logic from Heap::allocate to CollectorBitmap::advanceToNextPossibleFreeCell. Hey, I uploaded the patch to the correct bug this time!
Comment on attachment 57132 [details] Moved the advance logic from Heap::allocate to CollectorBitmap::advanceToNextPossibleFreeCell. r=me
Comment on attachment 57132 [details] Moved the advance logic from Heap::allocate to CollectorBitmap::advanceToNextPossibleFreeCell. Let's have auto-Eric take a crack at committing this.
Comment on attachment 57132 [details] Moved the advance logic from Heap::allocate to CollectorBitmap::advanceToNextPossibleFreeCell. Clearing flags on attachment: 57132 Committed r60323: <http://trac.webkit.org/changeset/60323>
All reviewed patches have been landed. Closing bug.