commit dccfb7c11fc933cb77ab19a2b3b9db63ee40032d
author Boyan Karatotev <boyan.karatotev@arm.com> Mon Apr 14 13:27:31 2025 +0100
committer Boyan Karatotev <boyan.karatotev@arm.com> Thu Jul 17 08:32:32 2025 +0100
tree 26c871a99bdcac5851b088fb0243446642e08a07
parent 000fe221b859ee82a4e2f8bf2c96f0086a772c89

refactor(build): simplify ENABLE_LTO checking

We do not support LTO with AArch32. The standard way to enforce this is
by having a rule to fail the build when this happens. There are no
AArch32 platforms which (incorrectly) try to enable LTO, so add this
rule. The benefit is that we no longer have to check the ENABLE_LTO with
ARCH whenever it is used.

Change-Id: Ic4086a1f5122269bda1d75bd4474b98fde35b5af
Signed-off-by: Boyan Karatotev <boyan.karatotev@arm.com>

make_helpers/constraints.mk

diff --git a/make_helpers/constraints.mk b/make_helpers/constraints.mk
index 9e2e221..594d77b 100644
--- a/make_helpers/constraints.mk
+++ b/make_helpers/constraints.mk
@@ -236,6 +236,9 @@
 
 # Ensure that no Aarch64-only features are enabled in Aarch32 build
 ifeq (${ARCH},aarch32)
+        ifneq (${ENABLE_LTO},0)
+                $(error "ENABLE_LTO is not supported with ARCH=aarch32")
+        endif
 
 	# SME/SVE only supported on AArch64
 	ifneq (${ENABLE_SME_FOR_NS},0)