alan woodland wrote (19 January 2010 09:55): > I think there might be a problem in that patch still... > > On the Debian machine it built on last night __ARM_ARCH_4__ > was not defined, __ARM_ARCH_4T__ was defined, and __thumb__ > wasn't defined. > This meant that it ended up using the ARMv5 of the #else, > which used the blx instruction and failed to build because of > it. > > I think changing it to read: > #if defined(__ARM_ARCH_4__) || defined(__ARM_ARCH_4T__) > should fix things. Does that seem sane to you? > > Thanks, > Alan
Yes, that makes sense. A modified, slightly tidied-up patch which I prepared yesterday is attached here. For some reason, I thought I'd already uploaded it for some reason, but it looks I didn't... ** Attachment added: "Updated patch which copes with -march=armv4t" http://launchpadlibrarian.net/38079473/cr_atomic_arm.diff -- fails to build on armel/lucid https://bugs.launchpad.net/bugs/503185 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
