Hi Carsten! The architectures in question don’t use the JIT, that’s why they can all use a common header for the atomics code.
Regarding the non-JIT architectures, there was only one change and that was the renaming of the generic headers from -sparc.h and -ppc.h to -feeling-lucky.h. All the non-JIT archs can use the -feeling-lucky.h header now which is identical to -sparc.h and -ppc.h. Adrian > On Jun 21, 2017, at 7:30 PM, Carsten Schoenert <c.schoen...@t-online.de> > wrote: > > Hello Adrian, > >> On Mon, Jun 19, 2017 at 12:44:49AM +0200, John Paul Adrian Glaubitz wrote: >> ... >> Hmm, looks like I was mislead by the gitweb view. Looking at the >> source, AtomicOperations-ppc.h is included for everything but >> s390x. >> >> So, it's safe to assume it FTBFS on s390x because of the missing >> atomic operations. No idea about the other architectures though, >> it should definitely work. > > correct, I've no idea why all these non-rc architectures currently > failing too. > > You pointed to a AtomicOperations.h that is based on Mozilla 55 with > some dramatically changes within the JIT stuff against 52.2.0 so I was > conservative and only picked a modification from Mike [1] for s390x that > should work for now. > > For more like pulling those changes to the JIT compiler the needed > adjustments to the patch queue afterwards we haven't enough resources > now. > > The Thunderbird guys preparing a small fixup release 52.2.1 an we will > probably delay a new upload until they release this version. > > [1] > https://anonscm.debian.org/cgit/pkg-mozilla/iceweasel.git/commit/?h=esr52/master&id=9d8d85289512755140a8b0d69467d58ab8b677fe > > Regards > Carsten
