[coreboot-gerrit] New patch to review for coreboot: d07eada armv7: mark EABI compatibility symbols as used

Isaac Christensen (isaac.christensen@se-eng.com) gerrit at coreboot.org
Wed Aug 13 01:12:26 CEST 2014


Isaac Christensen (isaac.christensen at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6640

-gerrit

commit d07eadaa366fa34065a6b815b6863371ca8fa9bf
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Tue Sep 10 11:02:58 2013 -0700

    armv7: mark EABI compatibility symbols as used
    
    These symbols are not used anywhere in our C code, so
    when using GCC's link time optimization feature they
    will be dropped even though they're needed by libgcc.
    Hence we need to mark them as used so GCC does not stumble
    and fall over its own guts.
    
    Change-Id: Ib2e9ea2610b57ab8244d5b699dd56025a4f08a01
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
    Reviewed-on: https://chromium-review.googlesource.com/168773
    (cherry picked from commit 416ffc880bcf4122b5430fbd9d9547c83886af2f)
    Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
---
 src/arch/armv7/eabi_compat.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/arch/armv7/eabi_compat.c b/src/arch/armv7/eabi_compat.c
index c063c85..0c79cd5 100644
--- a/src/arch/armv7/eabi_compat.c
+++ b/src/arch/armv7/eabi_compat.c
@@ -12,7 +12,7 @@
 #include <console/console.h>
 
 /* FIXME(dhendrix): prototypes added for assembler */
-int raise (int signum);
+int raise (int signum) __attribute__((used));
 int raise (int signum)
 {
 	printk(BIOS_CRIT, "raise: Signal # %d caught\n", signum);
@@ -20,12 +20,12 @@ int raise (int signum)
 }
 
 /* Dummy function to avoid linker complaints */
-void __aeabi_unwind_cpp_pr0(void);
+void __aeabi_unwind_cpp_pr0(void) __attribute__((used));
 void __aeabi_unwind_cpp_pr0(void)
 {
 };
 
-void __aeabi_unwind_cpp_pr1(void);
+void __aeabi_unwind_cpp_pr1(void) __attribute__((used));
 void __aeabi_unwind_cpp_pr1(void)
 {
 };



More information about the coreboot-gerrit mailing list