diff options
| author | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2015-07-07 00:26:59 +0200 |
|---|---|---|
| committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2015-07-07 17:08:16 +0200 |
| commit | 8ef928af4ed7dd0a42f40017aa4afb6a86575511 (patch) | |
| tree | e331db5db7d626075136570ead4dc16b6d7e5977 | |
| parent | e9e6e3d93c67c0e5df1b32fe37b1956624503c97 (diff) | |
xcompile: Fix compiler invocation in testcc
While for GCC targets the compiler is just defined as a single
binary, for clang it is defined as a binary and some options, e.g.:
clang -target i386-elf -ccc-gcc-name i386-elf-gcc
When executing the compiler with "$1", the shell will look for a
binary with the above name (instead of just clang) and always fail
detection of any CFLAGS.
By adding -c we prevent the compiler from failing because it can't
link a user space program (when what we're looking for, is whether
a specific compiler flag can be used to compile a coreboot object
file)
Change-Id: I1e9ff32fe40efbe3224c69785f31bc277f21d21b
Signed-off-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Reviewed-on: http://review.coreboot.org/10816
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
| -rwxr-xr-x | util/xcompile/xcompile | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile index c671172da1..7ae1e39cc9 100755 --- a/util/xcompile/xcompile +++ b/util/xcompile/xcompile @@ -72,7 +72,7 @@ testcc() { local tmp_o="$TMPFILE.o" rm -f "$tmp_c" "$tmp_o" echo "void _start(void) {}" >"$tmp_c" - "$1" -nostdlib -Werror $2 "$tmp_c" -o "$tmp_o" >/dev/null 2>&1 + $1 -nostdlib -Werror $2 -c "$tmp_c" -o "$tmp_o" >/dev/null 2>&1 } testas() { |