sched: Disable -Wunused-but-set-variable
authorPeter Zijlstra <peterz@infradead.org>
Tue, 12 Oct 2021 14:08:07 +0000 (16:08 +0200)
committerPeter Zijlstra <peterz@infradead.org>
Fri, 15 Oct 2021 09:25:15 +0000 (11:25 +0200)
The compilers can't deal with obvious DCE vs that warning, resulting
in code like:

if (0) {
sched sched_statistics *stats;

stats = __schedstats_from_se(se);

...
}

triggering the warning. Kill the warning to make the robots stop
reporting this.

Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
Link: https://lkml.kernel.org/r/YWWPLnaZGybHsTkv@hirez.programming.kicks-ass.net
kernel/sched/Makefile

index 978fcfc..c7421f2 100644 (file)
@@ -3,6 +3,10 @@ ifdef CONFIG_FUNCTION_TRACER
 CFLAGS_REMOVE_clock.o = $(CC_FLAGS_FTRACE)
 endif
 
+# The compilers are complaining about unused variables inside an if(0) scope
+# block. This is daft, shut them up.
+ccflags-y += $(call cc-disable-warning, unused-but-set-variable)
+
 # These files are disabled because they produce non-interesting flaky coverage
 # that is not a function of syscall inputs. E.g. involuntary context switches.
 KCOV_INSTRUMENT := n