[Skiboot] [PATCH 00/13] Support building with clang
Balbir Singh
bsingharora at gmail.com
Wed May 2 19:05:20 AEST 2018
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
Balbir Singh.
More information about the Skiboot
mailing list