diff options
author | Nick Desaulniers <nick.desaulniers@gmail.com> | 2017-10-07 16:23:23 -0400 |
---|---|---|
committer | Masahiro Yamada <yamada.masahiro@socionext.com> | 2017-10-26 10:58:45 -0400 |
commit | df16aaac26e92e97ab7234d3f93c953466adc4b5 (patch) | |
tree | ff138fd5db1d02cec9c97c0f31026f3ec3db8f3a /Makefile | |
parent | 2c1f4f125159f10521944cea23e33a00fcf85ede (diff) |
kbuild: clang: remove crufty HOSTCFLAGS
When compiling with `make CC=clang HOSTCC=clang`, I was seeing warnings
that clang did not recognize -fno-delete-null-pointer-checks for HOSTCC
targets. These were added in commit 61163efae020 ("kbuild: LLVMLinux:
Add Kbuild support for building kernel with Clang").
Clang does not support -fno-delete-null-pointer-checks, so adding it to
HOSTCFLAGS if HOSTCC is clang does not make sense.
It's not clear why the other warnings were disabled, and just for
HOSTCFLAGS, but I can remove them, add -Werror to HOSTCFLAGS and compile
with clang just fine.
Suggested-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Signed-off-by: Nick Desaulniers <nick.desaulniers@gmail.com>
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Diffstat (limited to 'Makefile')
-rw-r--r-- | Makefile | 5 |
1 files changed, 0 insertions, 5 deletions
@@ -365,11 +365,6 @@ HOSTCXXFLAGS := -O2 $(HOST_LFS_CFLAGS) | |||
365 | HOSTLDFLAGS := $(HOST_LFS_LDFLAGS) | 365 | HOSTLDFLAGS := $(HOST_LFS_LDFLAGS) |
366 | HOST_LOADLIBES := $(HOST_LFS_LIBS) | 366 | HOST_LOADLIBES := $(HOST_LFS_LIBS) |
367 | 367 | ||
368 | ifeq ($(shell $(HOSTCC) -v 2>&1 | grep -c "clang version"), 1) | ||
369 | HOSTCFLAGS += -Wno-unused-value -Wno-unused-parameter \ | ||
370 | -Wno-missing-field-initializers -fno-delete-null-pointer-checks | ||
371 | endif | ||
372 | |||
373 | # Make variables (CC, etc...) | 368 | # Make variables (CC, etc...) |
374 | AS = $(CROSS_COMPILE)as | 369 | AS = $(CROSS_COMPILE)as |
375 | LD = $(CROSS_COMPILE)ld | 370 | LD = $(CROSS_COMPILE)ld |