[coreboot-gerrit] New patch to review for coreboot: ca7897a xcompile: only enforce BFD linker on x86
Isaac Christensen (isaac.christensen@se-eng.com)
gerrit at coreboot.org
Sat Aug 9 01:27:47 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/6546
-gerrit
commit ca7897a763e7f88a2d4c605e4216baa63f26137e
Author: Stefan Reinauer <reinauer at chromium.org>
Date: Tue Sep 10 10:55:26 2013 -0700
xcompile: only enforce BFD linker on x86
The ARM linker scripts work fine with the gold linker.
This also requires to enhance the LINKER_SUFFIX variable
with a platform suffix so that it can be different on
ARMv7 and x86
Change-Id: I7d3b57991b1e40d0305be3fc4bc63d322392d98e
Signed-off-by: Stefan Reinauer <reinauer at google.com>
Reviewed-on: https://chromium-review.googlesource.com/168771
Reviewed-by: Ronald Minnich <rminnich at chromium.org>
Reviewed-by: Aaron Durbin <adurbin at chromium.org>
(cherry picked from commit 8b2f31e7b4996aeda5976d87971afe6cf56bef83)
Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
---
util/xcompile/xcompile | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index 01d75bf..21d8a3b 100755
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -125,9 +125,7 @@ detect_special_flags() {
testcc "$CC" "$CFLAGS -Wno-unused-but-set-variable " &&
CFLAGS="$CFLAGS -Wno-unused-but-set-variable "
- # Use bfd linker instead of gold if available:
- testcc "$CC" "$CFLAGS -fuse-ld=bfd" &&
- CFLAGS="$CFLAGS -fuse-ld=bfd" && LINKER_SUFFIX='.bfd'
+
testcc "$CC" "$CFLAGS -fno-stack-protector"&&
CFLAGS="$CFLAGS -fno-stack-protector"
@@ -138,6 +136,9 @@ detect_special_flags() {
x86)
testcc "$CC" "$CFLAGS -Wa,--divide" &&
CFLAGS="$CFLAGS -Wa,--divide"
+ # Use bfd linker instead of gold if available:
+ testcc "$CC" "$CFLAGS -fuse-ld=bfd" &&
+ CFLAGS="$CFLAGS -fuse-ld=bfd" && LINKER_SUFFIX='.bfd'
;;
esac
}
@@ -150,7 +151,7 @@ CC_${TARCH}:=${GCCPREFIX}gcc
CFLAGS_${TARCH}:=${CFLAGS}
CPP_${TARCH}:=${GCCPREFIX}cpp
AS_${TARCH}:=${GCCPREFIX}as ${ASFLAGS}
-LD_${TARCH}:=${GCCPREFIX}ld${LINKER_SUFFIX} ${LDFLAGS}
+LD_${TARCH}:=${GCCPREFIX}ld$(eval echo \${LINKER_SUFFIX_${TARCH}}) ${LDFLAGS}
NM_${TARCH}:=${GCCPREFIX}nm
OBJCOPY_${TARCH}:=${GCCPREFIX}objcopy
OBJDUMP_${TARCH}:=${GCCPREFIX}objdump
More information about the coreboot-gerrit
mailing list