the following patch was just integrated into master: commit 18607f717eb125f49431da3843c19fcb25f1ee81 Author: Patrick Georgi patrick@georgi-clan.de Date: Sun Oct 7 22:11:07 2012 +0200
Drop ramstage.a
ramstage.a has two issues: 1. duplicate source filenames don't survive the ar(1) treatment properly (so files aren't considered) 2. ld doesn't resolve symbols if it isn't forced to, in particular no overrides of weak symbols
Downside: The resulting binaries get slightly larger. Link time optimizations should fix that, as would tighter rules in the build system (to not compile unused code in the first place).
Change-Id: Iaae771ec8f92b42069237acd3b79c14e5bf9c03d Signed-off-by: Patrick Georgi patrick@georgi-clan.de Reviewed-on: http://review.coreboot.org/1566 Tested-by: build bot (Jenkins) Reviewed-by: Ronald G. Minnich rminnich@gmail.com
Build-Tested: build bot (Jenkins) at Tue Nov 27 21:56:45 2012, giving +1 Reviewed-By: Ronald G. Minnich rminnich@gmail.com at Tue Nov 27 21:59:25 2012, giving +2 See http://review.coreboot.org/1566 for details.
-gerrit