We should add support for MallocScribble to bmalloc.
<rdar://problem/29118340>
Created attachment 294968 [details] Patch v1
Comment on attachment 294968 [details] Patch v1 Why do we want to implement scribbling in bmalloc instead of deferring to system malloc? Do we also plan to duplicate the 14 other Malloc* options, or is there something special about scribbling? In general, I think it's best to defer debugging options to a separate implementation -- system malloc or our own -- so that debugging cruft doesn't sneak into high-performance code. For example, I don't think we want to do the m_isMallocScribbleEnabled check in deallocateFastCase. Even if we mark it unlikely, we still increase code size and harm register allocation. I went out of my way to avoid even checking m_isBmallocEnabled in deallocateFastCase. I'd like to keep that code as tight as possible.
Comment on attachment 294968 [details] Patch v1 Patches that have been up for review since 2016 are almost certainly too stale to be relevant to trunk in their current form. If this patch is still important please rebase it and post it for review again.