Re: [PATCH] kbuild: Add extra gcc checks

[Ingo Molnar]

* Borislav Petkov <bp@xxxxxxxxx> wrote:

> Add a 'W=1' Makefile switch which adds additional checking per build
> object.
> 
> The idea behind this option is targeted at developers who, in the
> process of writing their code, want to do the occasional
> 
> make W=1 [target.o]
> 
> and let gcc do more extensive code checking for them. Then, they
> could eyeball the output for valid gcc warnings about various
> bugs/discrepancies which are not reported during the normal build
> process.
> 
> For more background information and a use case, read through this
> thread: http://marc.info/?i=20110218091716.GA4384@bicker
> 
> Cc: Ingo Molnar <mingo@xxxxxxx>
> Cc: Michal Marek <mmarek@xxxxxxx>
> Cc: linux-kbuild@xxxxxxxxxxxxxxx
> Signed-off-by: Borislav Petkov <bp@xxxxxxxxx>

Nice.

Acked-by: Ingo Molnar <mingo@xxxxxxx>

We enable a lot of GCC warnings in tools/perf/ as well, and while there are false 
positives occasionally, the general effect on code quality is positive. We combine 
it with -Werror to make sure warnings do not accumulate.

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/