[coreboot-gerrit] Change in coreboot[master]: toolchain.inc: Use -Wstack-usage only on gcc

Patrick Georgi (Code Review) gerrit at coreboot.org
Wed May 10 22:09:44 CEST 2017


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 at 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),\

-- 
To view, visit https://review.coreboot.org/19657
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2c22171dedc77df24e739ec26335010f0f443963
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Patrick Georgi <pgeorgi at google.com>



More information about the coreboot-gerrit mailing list