Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/55549 )
Change subject: toolchain.inc: copy architecture specific CFLAGS to GCC_ADAFLAGS ......................................................................
toolchain.inc: copy architecture specific CFLAGS to GCC_ADAFLAGS
This fixes building with libgfxinit in x86_64 mode, which can generates unsupported R_X86_64_32S relocations without -mcmodel=large.
Change-Id: If5162fae475f3e70c856d9664a8cfc6a89d82283 Signed-off-by: Iru Cai mytbk920423@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/55549 Reviewed-by: Arthur Heymans arthur@aheymans.xyz Reviewed-by: Nico Huber nico.h@gmx.de Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M toolchain.inc 1 file changed, 7 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Nico Huber: Looks good to me, approved Arthur Heymans: Looks good to me, but someone else must approve
diff --git a/toolchain.inc b/toolchain.inc index 18c6ab2..80dab78 100644 --- a/toolchain.inc +++ b/toolchain.inc @@ -50,6 +50,13 @@ CFLAGS_x86_64 += -mcmodel=large -mno-red-zone CFLAGS_ppc64 +=
+GCC_ADAFLAGS_arm += +GCC_ADAFLAGS_arm64 += -mgeneral-regs-only +GCC_ADAFLAGS_riscv += +GCC_ADAFLAGS_x86_32 += +GCC_ADAFLAGS_x86_64 += -mcmodel=large -mno-red-zone +GCC_ADAFLAGS_ppc64 += + # Some boards only provide 2K stacks, so storing lots of data there leads to # problems. Since C rules don't allow us to statically determine the maximum # stack use, we use 1.5K as heuristic, assuming that we typically have lots