Author: hailfinger
Date: 2008-08-27 03:10:27 +0200 (Wed, 27 Aug 2008)
New Revision: 825
Modified:
coreboot-v3/Kconfig
coreboot-v3/arch/x86/Makefile
Log:
Enable compilation with -fwhole-program for initram. The setting can be
chosen via Kconfig. Default off.
Depending on the target, this saves between 10% and 20% of initram size.
Compilation still broken for:
- ADL MSM800SEV
- Gigabyte M57SLI
Compilation changed for:
- AMD DB800 (checker doesn't trigger anymore, but we get an unresolved
symbol instead)
- AMD Norwich (same)
(The compilation changes are not bad per se, just a bit misleading. It
seems that gcc 4.2.x is really smart. gcc 4.3.x and later are even
smarter and even fix compilation completely.)
Compilation OK for:
- AMD Serengeti
- Artec DBE61
- Artec DBE62
- PCEngines Alix.1C
- PCEngines Alix.2C3
-
Signed-off-by: Carl-Daniel Hailfinger <[EMAIL PROTECTED]>
Acked-by: Ronald G. Minnich <[EMAIL PROTECTED]>
Modified: coreboot-v3/Kconfig
===================================================================
--- coreboot-v3/Kconfig 2008-08-26 22:12:02 UTC (rev 824)
+++ coreboot-v3/Kconfig 2008-08-27 01:10:27 UTC (rev 825)
@@ -60,6 +60,22 @@
help
Enable this option to make coreboot beep upon certain events.
+config WHOLE_PROGRAM_COMPILE
+ bool "Enable whole-program optimizations (DANGEROUS)"
+ depends EXPERT && EXPERIMENTAL
+ help
+ Enable gcc -fwhole-program -combine for select code.
+ Needs lots of annotation in stage1 and stage2.
+ Works for initram only.
+ This increases compile time (no parallel compilation possible
+ anymore) and compiler memory usage (up to a factor of 20) and
+ makes debugging really hard.
+
+ Requires at least gcc 4.2.x, but miscompilations may occur.
+
+ Say No.
+ If you are a Gentoo user, say NO!
+
endmenu
source mainboard/Kconfig
Modified: coreboot-v3/arch/x86/Makefile
===================================================================
--- coreboot-v3/arch/x86/Makefile 2008-08-26 22:12:02 UTC (rev 824)
+++ coreboot-v3/arch/x86/Makefile 2008-08-27 01:10:27 UTC (rev 825)
@@ -132,6 +132,12 @@
PARSEELF = -e
endif
+ifeq ($(CONFIG_WHOLE_PROGRAM_COMPILE), y)
+ COMBINEFLAGS = -fwhole-program
+else
+ COMBINEFLAGS =
+endif
+
STAGE0_SRC := $(patsubst %,$(src)/lib/%,$(STAGE0_LIB_SRC)) \
$(patsubst %,$(src)/arch/x86/%,$(STAGE0_ARCH_X86_SRC)) \
$(STAGE0_MAINBOARD_SRC) $(STAGE0_CHIPSET_SRC)
@@ -268,7 +274,7 @@
$(obj)/coreboot.initram $(obj)/coreboot.initram.map: $(obj)/stage0.init
$(obj)/stage0-prefixed.o $(INITRAM_SRC)
$(Q)printf " CC $(subst $(shell pwd)/,,$(@)) (XIP)\n"
- $(Q)$(CC) $(INITCFLAGS) -fPIE -c -combine $(INITRAM_SRC) -o
$(obj)/coreboot.initram_partiallylinked.o
+ $(Q)$(CC) $(INITCFLAGS) -fPIE -c -combine $(COMBINEFLAGS)
$(INITRAM_SRC) -o $(obj)/coreboot.initram_partiallylinked.o
$(Q)# .data and .bss must be empty because they aren't handled
$(Q)printf " CHECK initram (non-empty writable/allocatable
sections)\n"
--
coreboot mailing list
[email protected]
http://www.coreboot.org/mailman/listinfo/coreboot
