[coreboot-gerrit] New patch to review for coreboot: e58f68f build system: only query the compiler runtime's location once

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon May 25 20:14:13 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10294

-gerrit

commit e58f68f6c7d20f2c84769c884724f4764f60157a
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Mon May 25 20:12:59 2015 +0200

    build system: only query the compiler runtime's location once
    
    No need to execute the compiler to figure this out once for each
    source file (or so).
    
    Change-Id: I56bf084f1217b96748296931617e9233f21183d5
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
---
 toolchain.inc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/toolchain.inc b/toolchain.inc
index de50471..e089f2e 100644
--- a/toolchain.inc
+++ b/toolchain.inc
@@ -106,9 +106,9 @@ CFLAGS_$(1) = $$(CFLAGS_common) $$(CFLAGS_$(2))
 CPPFLAGS_$(1) = $$(CPPFLAGS_common) $$(CPPFLAGS_$(2))
 COMPILER_RT_FLAGS_$(1) :=
 ifeq ($(CONFIG_COMPILER_LLVM_CLANG),y)
-COMPILER_RT_$(1) = $(shell $(CC_$(2)) $(CFLAGS_$(2)) -print-librt-file-name 2>/dev/null)
+COMPILER_RT_$(1) := $(shell $(CC_$(2)) $(CFLAGS_$(2)) -print-librt-file-name 2>/dev/null)
 else
-COMPILER_RT_$(1) = $(wildcard $(shell $(CC_$(2)) $(CFLAGS_$(2)) -print-libgcc-file-name))
+COMPILER_RT_$(1) := $(wildcard $(shell $(CC_$(2)) $(CFLAGS_$(2)) -print-libgcc-file-name))
 ifeq ($(2),x86_32)
 COMPILER_RT_FLAGS_$(1) := --wrap __divdi3 --wrap __udivdi3 --wrap __moddi3 --wrap __umoddi3
 endif



More information about the coreboot-gerrit mailing list