vendorcode/amd/agesa: Remove -fno-zero-initialized-in-bss
There are zero-initialized arrays within AGESA that were previously not
declared with CONST qualifier. Without this flag, such arrays would have
consumed valuable CAR space in romstage.
After adding CONST qualifiers these arrays have actually moved to
.rodata and removing the flag does not add anything to .bss.
TEST: see that BUILD_TIMELESS=1 results in the same binary.
Change-Id: I5b91deb1bf1b64bd9c88dc311db4e0b36df86c18
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/64445
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-by: Martin L Roth <gaumless@tutanota.com>
diff --git a/src/vendorcode/amd/agesa/Makefile.inc b/src/vendorcode/amd/agesa/Makefile.inc
index 73a87a3..9ef7f5c 100644
--- a/src/vendorcode/amd/agesa/Makefile.inc
+++ b/src/vendorcode/amd/agesa/Makefile.inc
@@ -9,7 +9,6 @@
classes-y += libagesa
libagesa-y =
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