diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2022-05-18 09:30:03 +0200 |
|---|---|---|
| committer | Martin L Roth <gaumless@tutanota.com> | 2022-05-28 04:21:54 +0000 |
| commit | 404188f80e7733610fd8d8611b55107f4e1f6431 (patch) | |
| tree | 965dcc53fbbadb31dada3b32b8f4efa22564fd4e | |
| parent | 9e9dccb89fe47af5f481b0c656399a61d5c4b79d (diff) | |
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>
| -rw-r--r-- | src/vendorcode/amd/agesa/Makefile.inc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/vendorcode/amd/agesa/Makefile.inc b/src/vendorcode/amd/agesa/Makefile.inc index 73a87a3c5a..9ef7f5cd19 100644 --- a/src/vendorcode/amd/agesa/Makefile.inc +++ b/src/vendorcode/amd/agesa/Makefile.inc @@ -9,7 +9,6 @@ subdirs-y += common 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 |