sysadmin/ipxe
1
0
Fork 0
mirror of https://github.com/ipxe/ipxe synced 2026-02-07 19:38:57 +03:00
Code Issues Packages Projects Releases Wiki Activity

[arm] Use -fno-short-enums for all 32-bit ARM builds

Browse Source 
The EFI ABI requires the use of -fno-short-enums, and the EDK2 headers
will perform a compile-time check that enums are 32 bits.

The EDK2 headers may be included even in builds for non-EFI platforms,
and so the -fno-short-enums flag must be used in all 32-bit ARM
builds.  Fortunately, nothing else currently cares about enum sizes.

Signed-off-by: Michael Brown <mcb30@ipxe.org>
This commit is contained in:
Michael Brown
2023-01-23 01:26:46 +00:00
parent 8f59911b20
commit 9de6c45dd3
2 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/arch/arm32/Makefile
View File

@@ -13,6 +13,11 @@ ASFLAGS += -mthumb -mcpu=cortex-a15
# #
CFLAGS += -fshort-wchar CFLAGS += -fshort-wchar
# EFI requires that enums are always 32 bits, and nothing else
# currently cares
#
CFLAGS += -fno-short-enums
# Include common ARM Makefile # Include common ARM Makefile
MAKEDEPS += arch/arm/Makefile MAKEDEPS += arch/arm/Makefile
include arch/arm/Makefile include arch/arm/Makefile

4
src/arch/arm32/Makefile.efi
View File

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