[coreboot-gerrit] New patch to review for coreboot: xcompile: Separate flags from clang executable

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Tue Jan 5 23:01:21 CET 2016


Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12840

-gerrit

commit 01c3d4229da98b4a7363e179c6b09d169465abde
Author: Martin Roth <martinroth at google.com>
Date:   Tue Jan 5 14:34:59 2016 -0700

    xcompile: Separate flags from clang executable
    
    We already have a CFLAGS variable - Use it for all of the flags.
    
    Change-Id: I22b4c5cf24b8743e85ffab29ddcccdc6c732ea3b
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 util/xcompile/xcompile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index bce8169..ebaf286 100755
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -350,7 +350,8 @@ test_architecture() {
 		# FIXME: this may break in a clang && !gcc configuration,
 		# but that's more of a clang limitation. Let's be optimistic
 		# that this will change in the future.
-		CLANG="${XGCCPATH}clang -target ${clang_arch}-${TABI} -ccc-gcc-name ${GCC}"
+		CLANG="${XGCCPATH}clang"
+		CFLAGS_CLANG="-target ${clang_arch}-${TABI} $CFLAGS_CLANG -ccc-gcc-name ${GCC}"
 	fi
 }
 



More information about the coreboot-gerrit mailing list