[coreboot-gerrit] [S] Change in coreboot[master]: util/xcompile: Fix building for 64bit

[Arthur Heymans (Code Review)]

Attention is currently required from: Martin L Roth.

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69232 )

Change subject: util/xcompile: Fix building for 64bit
......................................................................

util/xcompile: Fix building for 64bit

TESTED: This also boots on qemu q35

Change-Id: I7f99ebea18d5c09fdc7ced5c793d57d6fedd2e47
Signed-off-by: Arthur Heymans 
---
M util/xcompile/xcompile
1 file changed, 21 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/32/69232/1

diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index e35904f..e6d3123 100755
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -235,6 +235,13 @@
 GCC_ADAFLAGS_${TARCH}:=${FLAGS_GCC}
 GCC_COMPILER_RT_${TARCH}:=${CC_RT_GCC}
 GCC_COMPILER_RT_FLAGS_${TARCH}:=${CC_RT_EXTRA_GCC}
+EOF
+if [ "${TARCH}" = "x86_64" ]; then
+cat <<EOF
+	GCC_CFLAGS_${TARCH} += -malign-data=abi
+EOF
+fi
+cat <<EOF

# Clang
 CLANG_CC_${TARCH}:=${CLANG}
@@ -270,7 +277,7 @@
 # to use i586 instead.
 if [ "${TARCH}" = "x86_64" ]; then
 cat <<EOF
-	CFLAGS_${TARCH} += -march=nocona -malign-data=abi
+	CFLAGS_${TARCH} += -march=nocona
 EOF
 fi

@@ -459,7 +466,7 @@
    	# but that's more of a clang limitation. Let's be optimistic
    	# that this will change in the future.
    	CLANG="${clang_prefix}clang"
-		CFLAGS_CLANG="-target ${clang_arch}-${TABI} $CFLAGS_CLANG -ccc-gcc-name ${GCC}"
+		CFLAGS_CLANG="-target ${clang_arch}-${TABI} $CFLAGS_CLANG"
    fi
 }