From 8866c919f86ad2f00ff63192fa23cd19f749f90f Mon Sep 17 00:00:00 2001 From: Michael Brown Date: Fri, 28 Jul 2017 15:40:44 +0100 Subject: [PATCH] [build] Fix ARM32 EFI builds with current EDK2 headers 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 Signed-off-by: Michael Brown --- src/arch/arm32/Makefile.efi | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/arch/arm32/Makefile.efi b/src/arch/arm32/Makefile.efi index a06354f1..e139a055 100644 --- a/src/arch/arm32/Makefile.efi +++ b/src/arch/arm32/Makefile.efi @@ -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)