The bug noted in ToInt32 doesn't just exist when compiling for arm-darwin; it exists for ARM targets generally. The ifdef should reflect that.
Created attachment 8761217 [details] [diff] [review] adjust ifdefs in Conversions.h:ToInt32 to catch clang-on-arm-generally It's unfortunate that compiling with clang has to disable the fast path for this. clang also chokes on the inline assembly.
Attachment #8761217 - Flags: review?(luke)
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/bd25b06261c8 adjust ifdefs in Conversions.h:ToInt32 to catch clang-on-arm-generally; r=luke
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.