[PATCH] Makefile: 'Promote' -Wunused-but-set-variable to W=3

Subsystems: kernel build + files below scripts/ (unless maintained elsewhere), the rest

STALE1828d

2 messages, 2 authors, 2021-09-11 · open the first message on its own page

[PATCH] Makefile: 'Promote' -Wunused-but-set-variable to W=3

From: Peter Zijlstra <peterz@infradead.org>
Date: 2021-09-11 14:29:16

Upgrade -Wunused-but-set-variable to W=3 to avoid the build robots
sending endless spam for this garbage warning.

By promoting it to W=3 the robots will no longer report it and people
will no longer attempt to fix them by making the code worse.

Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
---
diff --git a/scripts/Makefile.extrawarn b/scripts/Makefile.extrawarn
index d53825503874..6686cc587d3b 100644
--- a/scripts/Makefile.extrawarn
+++ b/scripts/Makefile.extrawarn
@@ -28,7 +28,6 @@ KBUILD_CFLAGS += -Wmissing-format-attribute
 KBUILD_CFLAGS += -Wmissing-prototypes
 KBUILD_CFLAGS += -Wold-style-definition
 KBUILD_CFLAGS += -Wmissing-include-dirs
-KBUILD_CFLAGS += $(call cc-option, -Wunused-but-set-variable)
 KBUILD_CFLAGS += $(call cc-option, -Wunused-const-variable)
 KBUILD_CFLAGS += $(call cc-option, -Wpacked-not-aligned)
 KBUILD_CFLAGS += $(call cc-option, -Wstringop-truncation)
@@ -88,6 +87,7 @@ KBUILD_CFLAGS += -Wredundant-decls
 KBUILD_CFLAGS += -Wsign-compare
 KBUILD_CFLAGS += -Wswitch-default
 KBUILD_CFLAGS += $(call cc-option, -Wpacked-bitfield-compat)
+KBUILD_CFLAGS += $(call cc-option, -Wunused-but-set-variable)
 
 KBUILD_CPPFLAGS += -DKBUILD_EXTRA_WARN3
 

Re: [PATCH] Makefile: 'Promote' -Wunused-but-set-variable to W=3

From: Segher Boessenkool <hidden>
Date: 2021-09-11 21:37:15

On Sat, Sep 11, 2021 at 04:29:05PM +0200, Peter Zijlstra wrote:
Upgrade -Wunused-but-set-variable to W=3 to avoid the build robots
sending endless spam for this garbage warning.
"This warning for garbage code" you mean?
By promoting it to W=3 the robots will no longer report it and people
will no longer attempt to fix them by making the code worse.
Instead, they will not see such warnings, and make the code worse.


Segher
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help