[U-Boot] [PATCH v3 0/8] add clang support for some ARM boards

Jeroen Hofstee
2014-09-11T19:36:11Z

Hello Albert,

On 11-09-14 17:43, Albert ARIBAUD wrote:
> Hi Jeroen,
>
> Correction on the asm stuff:
>
> On Thu, 11 Sep 2014 13:17:20 +0200, Jeroen Hofstee
> <jeroen@myspectrum.nl> wrote:
>
>>> clang errors on arch/arm/lib/cache.c:28 for this:
>>> asm("0: mrc p15, 0, r15, c7, c10, 3\n\t" "bne 0b\n" : : : "memory");
>>> and that is a clang mistake, as for ARM926EJS r15 is a valid (albeit
>>> quite special semantically) Rd for Test and Clean DCache, see page 2-24.
>>>
>> This is the integrated-as complaining (the README tells you to disable
>> it for the moment). The clang folks push UAL hard, up to a point we need
>> to think about minimum gcc version etc. To avoid that, I just left out
>> such changes and just use gas instead, at least for the time being.
>> Below are some changes to compile versatileqemu with llvm integrated-as
>> and gcc/gas. No idea if it actually boots though.
> Actually, I had the -no-integrated-as then and have just re-tested now,
> making sure I have it and get the error above. For some reason, despite
> the -no-integrated-as option, the internal assembler is invoked.
>
>

You don't happen to be testing with the clang 3.5 minus a half /
non release (svn 201651) right? As I mentioned before, it will do
you more harm then good. I cannot reproduce this with an 3.4 nor
3.5 release.

Regards,
Jeroen