Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/69662 )
Change subject: soc/amd/common: Don't set gcc specific options for clang builds ......................................................................
soc/amd/common: Don't set gcc specific options for clang builds
Clang doesn't understand the -Wstack-usage=40960 option. Replace it with -Wframe-larger-than=40960.
Signed-off-by: Martin Roth gaumless@gmail.com Change-Id: I7d8b9c26d3fc861615a8553332ed1070974b751b Reviewed-on: https://review.coreboot.org/c/coreboot/+/69662 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Arthur Heymans arthur@aheymans.xyz Reviewed-by: Fred Reitberger reitbergerfred@gmail.com Reviewed-by: Elyes Haouas ehaouas@noos.fr Reviewed-by: Felix Held felix-coreboot@felixheld.de --- M src/soc/amd/common/psp_verstage/Makefile.inc 1 file changed, 23 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Felix Held: Looks good to me, approved Elyes Haouas: Looks good to me, approved Arthur Heymans: Looks good to me, approved Fred Reitberger: Looks good to me, approved
diff --git a/src/soc/amd/common/psp_verstage/Makefile.inc b/src/soc/amd/common/psp_verstage/Makefile.inc index fb82ef4..54e1d0a 100644 --- a/src/soc/amd/common/psp_verstage/Makefile.inc +++ b/src/soc/amd/common/psp_verstage/Makefile.inc @@ -4,8 +4,12 @@ verstage-generic-ccopts += -D__USER_SPACE__ CPPFLAGS_common += -I$(VBOOT_SOURCE)/firmware/2lib/include/
+ifeq ($(CONFIG_COMPILER_GCC),y) # This size should match the size in the linker script. CFLAGS_arm += -Wstack-usage=40960 +else +CFLAGS_arm += -Wframe-larger-than=40960 +endif
verstage-y += boot_dev.c verstage-y += delay.c