[webkit-reviews] review granted: [Bug 122460] Transition op_new_* JITStubs to JIT operations : [Attachment 214003] patch 2: with Michael's feedback applied.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Oct 11 11:36:53 PDT 2013


Michael Saboff <msaboff at apple.com> has granted Mark Lam <mark.lam at apple.com>'s
request for review:
Bug 122460: Transition op_new_* JITStubs to JIT operations
https://bugs.webkit.org/show_bug.cgi?id=122460

Attachment 214003: patch 2: with Michael's feedback applied.
https://bugs.webkit.org/attachment.cgi?id=214003&action=review

------- Additional Comments from Michael Saboff <msaboff at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=214003&action=review


r+

> Source/JavaScriptCore/jit/JITInlines.h:1068
> +#undef EABI_32BIT_DUMMY_ARG
> +#undef SH4_32BIT_DUMMY_ARG

This should be inside the #if USE(JSVALUE32_64) / #endif.

Move back to ToT placement for these defines. This puts the #define / #undef
immediately around the use.


More information about the webkit-reviews mailing list