Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d0eaa92d authored by Prasad Sodagudi's avatar Prasad Sodagudi
Browse files

kbuild: clang: disable unused variable warnings only when constant 



Currently, GCC disables -Wunused-const-variable, but not
-Wunused-variable, so warns unused variables if they are
non-constant.

While, Clang does not warn unused variables at all regardless of
the const qualifier because -Wno-unused-const-variable is implied
by the stronger option -Wno-unused-variable.

Disable -Wunused-const-variable instead of -Wunused-variable so that
GCC and Clang work in the same way.

Change-Id: I2e552288d4a9e19a15fd5e6f4d842e9304e69e28
Signed-off-by: default avatarPrasad Sodagudi <psodagud@codeaurora.org>
parent 64dad28d

Makefile

+1 −1
Original line number Diff line number Diff line
@@ -713,7 +713,6 @@ endif 
KBUILD_CFLAGS += $(CLANG_TARGET) $(CLANG_GCC_TC) -meabi gnu

KBUILD_AFLAGS += $(CLANG_TARGET) $(CLANG_GCC_TC)

KBUILD_CPPFLAGS += $(call cc-option,-Qunused-arguments,)

KBUILD_CFLAGS += $(call cc-disable-warning, unused-variable)

KBUILD_CFLAGS += $(call cc-disable-warning, format-invalid-specifier)

KBUILD_CFLAGS += $(call cc-disable-warning, gnu)

KBUILD_CFLAGS += $(call cc-disable-warning, address-of-packed-member)
@@ -744,6 +743,7 @@ KBUILD_CFLAGS += $(call cc-disable-warning, unused-but-set-variable) 
KBUILD_CFLAGS += $(call cc-disable-warning, unused-const-variable)

endif



KBUILD_CFLAGS += $(call cc-disable-warning, unused-const-variable)

ifdef CONFIG_FRAME_POINTER

KBUILD_CFLAGS	+= -fno-omit-frame-pointer -fno-optimize-sibling-calls

else