[coreboot-gerrit] Change in coreboot[master]: build system: Rely on xcompile for HOSTCC and HOSTCXX

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/43145 )

Change subject: build system: Rely on xcompile for HOSTCC and HOSTCXX
......................................................................

build system: Rely on xcompile for HOSTCC and HOSTCXX

It already looks for them, so let's use the result instead
of blindly defaulting to gcc/g++, except when not building an image
(but run kconfig or tests) because we don't use xcompile in those cases.

Change-Id: I3e50c70a609f1903a925610928f8779c191040d8
Signed-off-by: Patrick Georgi 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43145
Tested-by: build bot (Jenkins) 
Reviewed-by: Nico Huber 
---
M Makefile
1 file changed, 5 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, approved

diff --git a/Makefile b/Makefile
index 6a96f9d..2705c66 100644
--- a/Makefile
+++ b/Makefile
@@ -61,8 +61,6 @@
 # Disable implicit/built-in rules to make Makefile errors fail fast.
 .SUFFIXES:

-HOSTCC := $(if $(shell type gcc 2>/dev/null),gcc,cc)
-HOSTCXX = g++
 HOSTCFLAGS := -g
 HOSTCXXFLAGS := -g

@@ -128,6 +126,11 @@
    rm -f $@.tmp

ifeq ($(NOCOMPILE),1)
+# We also don't use .xcompile in the no-compile situations, so
+# provide some reasonable defaults.
+HOSTCC ?= $(if $(shell type gcc 2>/dev/null),gcc,cc)
+HOSTCXX ?= g++
+
 include $(TOPLEVEL)/Makefile.inc
 include $(TOPLEVEL)/payloads/Makefile.inc
 include $(TOPLEVEL)/util/testing/Makefile.inc