Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63040 )
Change subject: vendorcode/amd/agesa: Add CFLAGS required by CLANG ......................................................................
vendorcode/amd/agesa: Add CFLAGS required by CLANG
Vendorcode is messy so instead of trying to fix the warnings thrown by clang ignore them on AGESA platforms.
Change-Id: I378571c2b7272901761c786c6daec0a403155d4c Signed-off-by: Arthur Heymans arthur@aheymans.xyz Reviewed-on: https://review.coreboot.org/c/coreboot/+/63040 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Elyes Haouas ehaouas@noos.fr --- M src/vendorcode/amd/agesa/Makefile.inc 1 file changed, 5 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Elyes Haouas: Looks good to me, approved
diff --git a/src/vendorcode/amd/agesa/Makefile.inc b/src/vendorcode/amd/agesa/Makefile.inc index b96af84..73a87a3 100644 --- a/src/vendorcode/amd/agesa/Makefile.inc +++ b/src/vendorcode/amd/agesa/Makefile.inc @@ -11,6 +11,11 @@ libagesa-generic-ccopts += -D__LIBAGESA__ libagesa-generic-ccopts += -fno-zero-initialized-in-bss libagesa-generic-ccopts += $(AGESA_INC) $(AGESA_AUTOINCLUDES) +libagesa-generic-ccopts += -Wno-pragma-pack -Wno-unused-but-set-variable +libagesa-generic-ccopts += -Wno-implicit-fallthrough + +romstage-generic-ccopts += -Wno-pragma-pack +ramstage-generic-ccopts += -Wno-pragma-pack
AGESA_CFLAGS := -march=k8-sse3 -mtune=k8-sse3 -fno-strict-aliasing