Patrick Georgi has uploaded a new change for review. ( https://review.coreboot.org/19657 )
Change subject: toolchain.inc: Use -Wstack-usage only on gcc ......................................................................
toolchain.inc: Use -Wstack-usage only on gcc
clang isn't happy with it
Change-Id: I2c22171dedc77df24e739ec26335010f0f443963 Signed-off-by: Patrick Georgi pgeorgi@google.com --- M toolchain.inc 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/19657/1
diff --git a/toolchain.inc b/toolchain.inc index 5501c32..d9b11f0 100644 --- a/toolchain.inc +++ b/toolchain.inc @@ -79,11 +79,13 @@ # for an example). # (If you absolutely need a larger stack frame and are 100% sure it cannot # cause problems, you can whitelist it with #pragma diagnostic.) +ifeq ($(CONFIG_COMPILER_GCC),y) CFLAGS_arm += -Wstack-usage=1536 CFLAGS_arm64 += -Wstack-usage=1536 CFLAGS_mips += -Wstack-usage=1536 CFLAGS_riscv += -Wstack-usage=1536 CFLAGS_power8 += -Wstack-usage=1536 +endif
toolchain_to_dir = \ $(foreach arch,$(ARCH_SUPPORTED),\