]> git.ipfire.org Git - thirdparty/ipxe.git/commitdiff
[build] Fix ARM32 EFI builds with current EDK2 headers
authorMichael Brown <mcb30@ipxe.org>
Fri, 28 Jul 2017 14:40:44 +0000 (15:40 +0100)
committerMichael Brown <mcb30@ipxe.org>
Fri, 28 Jul 2017 14:46:06 +0000 (15:46 +0100)
EDK2 commit 6440385 ("MdePkg/Include: Add enumeration size checks to
Base.h") enforced the UEFI specification mandate that enums should
always be 32 bits.  This revealed a latent bug in iPXE, which does not
build with -fno-short-enums.

Fix by adding -fno-short-enums to CFLAGS for ARM32 EFI builds.

Reported-by: Benjamin S. Allen <bsallen@alcf.anl.gov>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
src/arch/arm32/Makefile.efi

index a06354f1d8fdd532d1085e79aa664369175accfa..e139a055a16c5b1a47437b059618f5849204a058 100644 (file)
@@ -1,5 +1,9 @@
 # -*- makefile -*- : Force emacs to use Makefile mode
 
+# UEFI requires that enums are always 32 bits
+#
+CFLAGS         += -fno-short-enums
+
 # Specify EFI image builder
 #
 ELF2EFI                = $(ELF2EFI32)