[Skiboot] [PATCH 00/13] Support building with clang
joel at jms.id.au
Thu May 3 13:04:45 AEST 2018
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
>> 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.
>> 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.
More information about the Skiboot