[coreboot] Patch merged into coreboot/master: 18607f7 Drop ramstage.a

gerrit at coreboot.org gerrit at coreboot.org
Tue Nov 27 21:59:26 CET 2012


the following patch was just integrated into master:
commit 18607f717eb125f49431da3843c19fcb25f1ee81
Author: Patrick Georgi <patrick at 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 at georgi-clan.de>
    Reviewed-on: http://review.coreboot.org/1566
    Tested-by: build bot (Jenkins)
    Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>

Build-Tested: build bot (Jenkins) at Tue Nov 27 21:56:45 2012, giving +1
Reviewed-By: Ronald G. Minnich <rminnich at gmail.com> at Tue Nov 27 21:59:25 2012, giving +2
See http://review.coreboot.org/1566 for details.

-gerrit




More information about the coreboot mailing list