[Skiboot] [PATCH 00/13] Support building with clang

[Joel Stanley]

On 2 May 2018 at 18:35, Balbir Singh <bsingharora at gmail.com> wrote:
> On Wed, 2018-05-02 at 18:07 +0930, Joel Stanley wrote:
>> Hello Skibooters!
>>
>> This series contains cleanups and modifications that are required to
>> build skiboot with clang.
>>
>> If you want to try it out:
>>
>>  $ sudo apt-get install clang
>>  $ make CC=clang
>>
>> I've done all of my testing with clang-7 (trunk). Clang 6 works fine
>> too, but earlier versions have a number of issues.
>>
>> Most of the patches could be considered cleanup or have no impact on GCC
>> builds.
>>
>> The last three that change the make file are quite clang specific, but
>> again have no impact on normal building of skiboot.
>>
>> Please test, and if they look good, merge! I have plans down the track
>> for using clang to improve our code quality.
>>
>> Cheers,
>>
>> Joel
>>
>> Joel Stanley (13):
>>   asm: Fix up assembly for clang
>>   cpu.h: Use this_cpu through accessor
>>   cpu.h: implement this_cpu for clang
>>   libflash/ecc: disable asm version for clang
>>   processor.h: implement sndmsg instructions
>>   Makefile: Use LD to link the final binary
>>   Fix asm-offsets generation
>>   pci-quirk: Fix initiliser warning
>>   imc: Remove extra parentheses in test
>>   fsp: Fix msg vaargs usage
>>   Makefile: Put some ppc options behind try-cflag tests
>>   Makefile: Add additional flags when using clang
>>   Makefile: Disable warnings to make clang happy
>>
>
> These patches don't seem to be in order, in the sense that I'd expect
> the first patch to fix the Makefile for -ffixed-reg and -m* and -Wl.
> Then fix the files that don't compile, etc

Yeah I had it that way, but thought this would make more sense. I'll
think a bit harder before posting a second version.

Cheers,

Joel