I hit a compile failure when compiling js 32-bit ARM-simulator shell on m-c rev 385840329d91. LD=ld CROSS_COMPILE=1 CC="clang -Qunused-arguments -msse2 -mfpmath=sse -arch i386" RANLIB=ranlib CXX="clang++ -Qunused-arguments -msse2 -mfpmath=sse -arch i386" AS=$CC AR=ar STRIP="strip -x -S" HOST_CC="clang -Qunused-arguments -msse2 -mfpmath=sse" AUTOCONF=/usr/local/Cellar/autoconf213/2.13/bin/autoconf213 HOST_CXX="clang++ -Qunused-arguments -msse2 -mfpmath=sse" sh /Users/skywalker/trees/mozilla-central/js/src/configure --target=i386-apple-darwin9.2.0 --enable-macos-target=10.5 --enable-arm-simulator --enable-debug --enable-optimize --enable-nspr-build --enable-more-deterministic --with-ccache --enable-gczeal --enable-debug-symbols --disable-tests === autoBisect shows this is probably related to the following changeset: The first bad revision is: changeset: https://hg.mozilla.org/mozilla-central/rev/b2904e8f07e7 user: Nicolas B. Pierron date: Sat Mar 28 01:08:12 2015 +0100 summary: Bug 1143011 - Extract the has/add/take logic out of the register sets to distinguish between allocatable and live sets. r=jandem,Waldo Nicolas, is bug 1143011 a likely regressor? This blocks fuzzing 32-bit ARM simulator builds on Mac.
The error message is typical of this patch, so yes, this is likely caused by this patch. On the other hand, I do not understand how we managed to have ARM specific code for Mac, as this builds correctly on Linux. Can you paste the compilation error that you see on master? The first part of Bug 1143011 is indeed likely to show a lot of errors as it removes function, but only the second patch change the uses.
Discussed with nbp on IRC, this one-liner fixes it for me: https://hg.mozilla.org/integration/mozilla-inbound/rev/814e2c0ad479
Assignee: nobody → jdemooij
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
We should backport this to aurora too.
(In reply to Gary Kwong [:gkw] [:nth10sd] from comment #5) > We should backport this to aurora too. Bug 1143011 was backed out from Aurora so we can't backport this. We should do that if/when we reland bug 1143011 though...
Ready for an Aurora nomination now.
Attachment #8599143 - Attachment description: Patch (ignore white space) → Patch
Comment on attachment 8599143 [details] [diff] [review] Patch Approval Request Comment [Feature/regressing bug #]: Bug 1143011. [User impact if declined]: Errors when compiling the ARM backend with Clang. [Describe test coverage new/current, TreeHerder]: No chance in behavior. Patch has been on m-c for weeks without problems. [Risks and why]: None; one-line fix. [String/UUID change made/needed]: None.
Attachment #8599143 - Flags: approval-mozilla-aurora?
(In reply to Jan de Mooij [:jandem] from comment #9) > No chance in behavior. Er, "change", of course.
Comment on attachment 8599143 [details] [diff] [review] Patch Approved for uplift to aurora.
Attachment #8599143 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
status-firefox39: affected → fixed
You need to log in before you can comment on or make changes to this bug.