On 20-Jul-16 16:19, Julius Werner wrote:
Well... honestly, isn't it our fault for uprevving to a buggy toolchain? If there was a way to write this code such that it will result in the binary output ARM wants and works on all versions, submitting a patch to them would be reasonable... but otherwise I can kinda understand that they don't want to use an inferior solution because we insist on using an assembler that doesn't know how to assemble stuff correctly. Isn't the best solution to only uprev archs that really need new compiler features until the GCC guys manage to produce a stable version?
Yes, I was wrong. After reading the other threads and talking with Martin, it indeed seems to be a toolchain issue. However, the new toolchain had many advantages over the previous one, and the code was not in (our) ARM TF at the time we uprevved.
The maintenance burden of keeping different versions of toolchains supported for different architectures as part of the reference toolchain is imho almost always significantly higher than actually fixing the issues. (Especially if they are not time critical)
Martin produced a binutils fix for the issue. Whether the binutils people will find it to be the "right" fix is still to be found out, but it seems to allow compiling the latest, unmodified ARM TF.
Stefan
Le mercredi 20 juillet 2016 à 19:33 -0700, Stefan Reinauer a écrit :
Martin produced a binutils fix for the issue. Whether the binutils people will find it to be the "right" fix is still to be found out, but it seems to allow compiling the latest, unmodified ARM TF.
Upstream just answered with a comprehensive description of what is happening and a patch to solve the issue, at: https://sourceware.org/bugzilla/show_bug.cgi?id=20364
Could someone try that and report back (either here directly on the tracker) whether it solves the issue? There's a chance this fix will be accepted upstream if it works.
Cheers,
Hey Paul, I'll get that patch tested and replace the current patch with it if it works.
Thanks, Martin
On Tue, Aug 2, 2016 at 9:51 AM, Paul Kocialkowski contact@paulk.fr wrote:
Le mercredi 20 juillet 2016 à 19:33 -0700, Stefan Reinauer a écrit :
Martin produced a binutils fix for the issue. Whether the binutils people will find it to be the "right" fix is still to be found out, but it seems to allow compiling the latest, unmodified ARM TF.
Upstream just answered with a comprehensive description of what is happening and a patch to solve the issue, at: https://sourceware.org/bugzilla/show_bug.cgi?id=20364
Could someone try that and report back (either here directly on the tracker) whether it solves the issue? There's a chance this fix will be accepted upstream if it works.
Cheers,
-- Paul Kocialkowski, developer of low-level free software for embedded devices
Website: https://www.paulk.fr/ Coding blog: https://code.paulk.fr/ Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/