pc-bios/s390-ccw: Allow building with Clang, too

[PATCH 0/4] pc-bios/s390-ccw: Allow building with Clang, too

Thomas Huth posted 4 patches 3 years ago

Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210502174836.838816-1-thuth@redhat.com

Maintainers: Cornelia Huck <cohuck@redhat.com>, Christian Borntraeger <borntraeger@de.ibm.com>, Thomas Huth <thuth@redhat.com>

configure                   | 9 ++++++++-
pc-bios/s390-ccw/Makefile   | 8 +++++---
pc-bios/s390-ccw/s390-ccw.h | 1 +
3 files changed, 14 insertions(+), 4 deletions(-)

Expand all Fold all

[PATCH 0/4] pc-bios/s390-ccw: Allow building with Clang, too

Posted by Thomas Huth 3 years ago

Clang can provide some additional warnings compared to GCC which can
sometimes help to catch some more bugs. So it would be good to be
able to build the s390-ccw bios with Clang, too. Only caveat: Clang
does not support the z900 anymore which is the lowest guest CPU that
could be used in QEMU, so when compiling the firmware with Clang,
it is only possible to use guest CPUs >= z10. Since the additional
compiler test coverage is worth the effort, allow it anyway and just
emit a warning in the configure step.

Philippe Mathieu-Daudé (1):
  pc-bios/s390-ccw: Silence GCC 11 stringop-overflow warning

Thomas Huth (3):
  pc-bios/s390-ccw: Silence warning from Clang by marking panic() as
    noreturn
  pc-bios/s390-ccw: Fix the cc-option macro in the Makefile
  pc-bios/s390-ccw: Allow building with Clang, too

 configure                   | 9 ++++++++-
 pc-bios/s390-ccw/Makefile   | 8 +++++---
 pc-bios/s390-ccw/s390-ccw.h | 1 +
 3 files changed, 14 insertions(+), 4 deletions(-)

-- 
2.27.0