Place the guard USE(JSVALUE32_64) inside the body of the DEFINE_STUB_FUNCTION macro. This I think makes the code a bit more readable as it is clear that DEFINE_STUB_FUNCTION(int, op_eq) is needed regardless of the USE(JSVALUE32_64) test. This refactoring fixes a build break for the RVCT tool chain. For the RVCT toolchain (e.g. for Symbian) precompiler macros are not expanded before the code is passed to the assembler. This tool chain limitation and the corresponding workaround (simple perl script as pre-compiler) been documented as part of commit r52970. The current perl script does not take all the C precompiler macros into account, so it expands the DEFINE_STUB_FUNCTION macro twice for DEFINE_STUB_FUNCTION(int, op_eq).
Created attachment 46953 [details] proposed build fix for RVCT
Looks sane, but I'll CC the JIT people for a real review.
Comment on attachment 46953 [details] proposed build fix for RVCT r=me
Comment on attachment 46953 [details] proposed build fix for RVCT Clearing flags on attachment: 46953 Committed r53789: <http://trac.webkit.org/changeset/53789>
All reviewed patches have been landed. Closing bug.