Message ID | 20211203132607.2648640-2-ross.burton@arm.com |
---|---|
State | New |
Headers | show |
Series | [v2,1/3] arm/edk2-firmware: correctly set the target prefix in Clang builds | expand |
diff --git a/meta-arm/recipes-bsp/uefi/edk2-firmware.inc b/meta-arm/recipes-bsp/uefi/edk2-firmware.inc index cfe30652..d8c744df 100644 --- a/meta-arm/recipes-bsp/uefi/edk2-firmware.inc +++ b/meta-arm/recipes-bsp/uefi/edk2-firmware.inc @@ -26,6 +26,10 @@ COMPATIBLE_MACHINE ?= "invalid" PACKAGE_ARCH = "${MACHINE_ARCH}" EDK2_BUILD_RELEASE = "1" +# Release builds don't work with clang yet: +# https://bugzilla.tianocore.org/show_bug.cgi?id=3758 +EDK2_BUILD_RELEASE:toolchain-clang = "0" + EDK2_PLATFORM = "unset" # build --platform EDK2_PLATFORM_DSC = "unset"
edk2-firmware release builds with clang fail: MdePkg/Library/UefiMemoryAllocationLib/MemoryAllocationLib.c:142:15: error: variable 'Status' set but not used [-Werror,-Wunused-but-set-variable] This is upstream as https://bugzilla.tianocore.org/show_bug.cgi?id=3758, but until that is resolved we can just force debug builds with clang. Signed-off-by: Ross Burton <ross.burton@arm.com> --- meta-arm/recipes-bsp/uefi/edk2-firmware.inc | 4 ++++ 1 file changed, 4 insertions(+)