It seems that GCC's LTO doesn't like the way we implement DECLARE_OPTIONAL_REGION(). This patch changes it so that rather than having a normal DECLARE_REGION() in <symbols.h> and then an extra DECLARE_OPTIONAL_REGION() in the C file using it, you just say DECLARE_OPTIONAL_REGION() directly in <symbols.h> (in place and instead of the usual DECLARE_REGION()). This basically looks the same way in the resulting object file but somehow LTO seems to like it better. Signed-off-by: Julius Werner <jwerner@chromium.org> Change-Id: I6096207b311d70c8e9956cd9406bec45be04a4a2 Reviewed-on: https://review.coreboot.org/c/coreboot/+/44791 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Jacob Garber <jgarber1@ualberta.ca> Reviewed-by: Hung-Te Lin <hungte@chromium.org> Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr> |
||
|---|---|---|
| .. | ||
| include | ||
| arch_timer.c | ||
| boot.c | ||
| bootblock.S | ||
| fit_payload.c | ||
| fp_asm.S | ||
| Kconfig | ||
| Makefile.inc | ||
| mcall.c | ||
| misaligned.c | ||
| misc.c | ||
| opensbi.c | ||
| payload.c | ||
| pmp.c | ||
| ramstage.S | ||
| romstage.c | ||
| sbi.c | ||
| smp.c | ||
| tables.c | ||
| trap_handler.c | ||
| trap_util.S | ||
| virtual_memory.c | ||