[U-Boot] [PATCH v2] kbuild: do not hide clang warnings during build with W=1

These lines originate in Linux; it looks like Linux folks chose to hide bunch of warnings reported by clang rather than fixing C code.
In U-boot, warnings are being fixed thanks to Jeroen's great efforts. Let's stop suppressing clang warnings.
Signed-off-by: Masahiro Yamada yamada.m@jp.panasonic.com Cc: Jeroen Hofstee jeroen@myspectrum.nl ---
Changes in v2: - Rephrase commit summary and description for clarification
scripts/Makefile.extrawarn | 10 ---------- 1 file changed, 10 deletions(-)
diff --git a/scripts/Makefile.extrawarn b/scripts/Makefile.extrawarn index 6564350..0ec0d24 100644 --- a/scripts/Makefile.extrawarn +++ b/scripts/Makefile.extrawarn @@ -26,16 +26,6 @@ warning-1 += $(call cc-option, -Wmissing-include-dirs) warning-1 += $(call cc-option, -Wunused-but-set-variable) warning-1 += $(call cc-disable-warning, missing-field-initializers)
-# Clang -warning-1 += $(call cc-disable-warning, initializer-overrides) -warning-1 += $(call cc-disable-warning, unused-value) -warning-1 += $(call cc-disable-warning, format) -warning-1 += $(call cc-disable-warning, unknown-warning-option) -warning-1 += $(call cc-disable-warning, sign-compare) -warning-1 += $(call cc-disable-warning, format-zero-length) -warning-1 += $(call cc-disable-warning, uninitialized) -warning-1 += $(call cc-option, -fcatch-undefined-behavior) - warning-2 := -Waggregate-return warning-2 += -Wcast-align warning-2 += -Wdisabled-optimization

On Fri, Oct 31, 2014 at 11:00:34AM +0900, Masahiro Yamada wrote:
These lines originate in Linux; it looks like Linux folks chose to hide bunch of warnings reported by clang rather than fixing C code.
In U-boot, warnings are being fixed thanks to Jeroen's great efforts. Let's stop suppressing clang warnings.
Signed-off-by: Masahiro Yamada yamada.m@jp.panasonic.com Cc: Jeroen Hofstee jeroen@myspectrum.nl
Applied to u-boot/master, thanks!
participants (2)
-
Masahiro Yamada
-
Tom Rini