[coreboot-gerrit] Patch merged into coreboot/master: 27cf247 car globals: add "used" attribute

gerrit at coreboot.org gerrit at coreboot.org
Sun Nov 9 12:25:59 CET 2014


the following patch was just integrated into master:
commit 27cf24727c26874cbe61734d01074f6da077d1b9
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Sat Nov 8 13:16:46 2014 +0100

    car globals: add "used" attribute
    
    Otherwise clang feels free to optimize away that variable
    (somewhat) and revive it in a different form inside .bss.
    
    They probably have the language lawyery excuse for why
    that's perfectly legal, so let's play it safe.
    (relevant URL, sorry ron: http://llvm.org/bugs/show_bug.cgi?id=9520)
    
    Change-Id: I603312ceea7207088dd29453cc8fb8f48c31af21
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
    Reviewed-on: http://review.coreboot.org/7357
    Reviewed-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
    Tested-by: build bot (Jenkins)


See http://review.coreboot.org/7357 for details.

-gerrit



More information about the coreboot-gerrit mailing list