[coreboot-gerrit] Patch set updated for coreboot: 946b86b build: break compiler flags out of $(CC)

Patrick Georgi (patrick@georgi-clan.de) gerrit at coreboot.org
Sat May 17 16:03:02 CEST 2014


Patrick Georgi (patrick at georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5769

-gerrit

commit 946b86beecfed3aa3636a3fc893729d6784edb0f
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Sat May 17 15:08:47 2014 +0200

    build: break compiler flags out of $(CC)
    
    Having more than the executable in $(CC) only leads to
    trouble in a number of situations.
    
    Change-Id: I7642ca4068b3a3bd5798219d74de9e0eb85bb4e5
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
---
 src/arch/x86/Makefile.inc | 2 +-
 toolchain.inc             | 4 ++--
 util/xcompile/xcompile    | 3 ++-
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/arch/x86/Makefile.inc b/src/arch/x86/Makefile.inc
index 30f9243..6c7008c 100644
--- a/src/arch/x86/Makefile.inc
+++ b/src/arch/x86/Makefile.inc
@@ -28,7 +28,7 @@ subdirs-y += lib
 subdirs-y += smp
 
 ifeq ($(CONFIG_COMPILER_GCC),y)
-DISASSEMBLY=-Wa,-acdlns
+DISASSEMBLY=-Wa,-acdlns $(CFLAGS_x86_32)
 else
 DISASSEMBLY=
 endif
diff --git a/toolchain.inc b/toolchain.inc
index b615bce..568b3d6 100644
--- a/toolchain.inc
+++ b/toolchain.inc
@@ -25,7 +25,7 @@ ARCHDIR-i386    := x86
 ARCHDIR-x86_32  := x86
 ARCHDIR-armv7   := armv7
 
-CFLAGS_armv7 = \
+CFLAGS_armv7 += \
 	-ffixed-r8\
 	-march=armv7-a\
 	-marm\
@@ -35,7 +35,7 @@ CFLAGS_armv7 = \
 
 toolchain_to_dir = \
 	$(foreach arch,$(ARCH_SUPPORTED),\
-	$(eval CPPFLAGS_$(arch) = \
+	$(eval CPPFLAGS_$(arch) += \
 	-Isrc/arch/$(ARCHDIR-$(arch))/include))
 
 # set_stage_toolchain: Decides the toolchain to be used by every stage
diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index b342de3..d133bd2 100644
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -147,7 +147,8 @@ detect_special_flags() {
 report_arch_toolchain() {
 	cat <<EOF
 # elf${TWIDTH}-${TBFDARCH} toolchain (${GCCPREFIX}gcc)
-CC_${TARCH}:=${GCCPREFIX}gcc ${CFLAGS}
+CC_${TARCH}:=${GCCPREFIX}gcc
+CFLAGS_${TARCH}:=${CFLAGS}
 CPP_${TARCH}:=${GCCPREFIX}cpp
 AS_${TARCH}:=${GCCPREFIX}as ${ASFLAGS}
 LD_${TARCH}:=${GCCPREFIX}ld${LINKER_SUFFIX} ${LDFLAGS}



More information about the coreboot-gerrit mailing list