[webkit-reviews] review granted: [Bug 132137] Make slowPathAllocsBetweenGCs a runtime option : [Attachment 230103] revised patch

[bugzilla-daemon at webkit.org]

Mark Hahnenberg <mhahnenberg at apple.com> has granted Mark Lam
<mark.lam at apple.com>'s request for review:
Bug 132137: Make slowPathAllocsBetweenGCs a runtime option
https://bugs.webkit.org/show_bug.cgi?id=132137

Attachment 230103: revised patch
https://bugs.webkit.org/attachment.cgi?id=230103&action=review

------- Additional Comments from Mark Hahnenberg <mhahnenberg at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=230103&action=review


r=me

> Source/JavaScriptCore/heap/MarkedAllocator.h:53
> +    ALWAYS_INLINE void doTestCollectionsIfNeeded();

Do you need ALWAYS_INLINE here? I've never seen it in a header before.