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

Joel Stanley joel at jms.id.au
Wed May 2 18:37:40 AEST 2018


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

 Makefile.main       | 44 ++++++++++++++++++++++++++++++++++++++------
 asm/head.S          |  4 ++--
 core/pci-quirk.c    |  2 +-
 hw/fsp/fsp.c        |  4 ++--
 hw/imc.c            |  2 +-
 include/cpu.h       | 24 +++++++++++++++++++++++-
 include/fsp.h       |  4 ++--
 include/processor.h | 33 ++++++++++++++++++++++++++++++---
 libflash/ecc.c      |  2 +-
 make_offsets.sh     |  2 +-
 10 files changed, 101 insertions(+), 20 deletions(-)

-- 
2.17.0



More information about the Skiboot mailing list