On Thu, 26 Oct 2017, Chris Wilson <[email protected]> wrote: > Quoting Jani Nikula (2017-10-26 15:36:34) >> On Tue, 24 Oct 2017, Chris Wilson <[email protected]> wrote: >> > Recently W=1 on gcc-7.2 (-Wunused-const-variable) caught a regression >> > that had been lurking for 6 months, so lets try enabling the full set of >> > warnings for CI builds. This means more patches will be rejected early >> > that contain trivial and sometimes not so trivial bugs. However, our >> > code does not yet compile cleanly with W=1, so we have to apply a filter >> > to the set of warnings until we can eliminate the mistakes. It also >> > means that developers will have to be running the full gamut of gcc to >> > ensure that as warnings come and go with gcc updates, we have the CI >> > build prepared. >> > >> > v2: Use fine-grained -Wno overrides. Inside the makefile, we can >> > specify CFLAGS on a per-object level, which allows us to limit the scope >> > of any particular warning override. >> > v3: Place per-file overrides after the main enabling block. >> > >> > Signed-off-by: Chris Wilson <[email protected]> >> > Cc: Joonas Lahtinen <[email protected]> >> > Cc: Jani Nikula <[email protected]> >> > Cc: Daniel Vetter <[email protected]> >> > Cc: Tomi Sarvela <[email protected]> >> > Cc: Michal Wajdeczko <[email protected]> >> > Cc: Ville Syrjälä <[email protected]> >> > Acked-by: Tomi Sarvela <[email protected]> >> > Reviewed-by: Joonas Lahtinen <[email protected]> >> > --- >> > Seeking more acks for making our lives harder by giving gcc free reign >> > in its warnings. >> >> Acked-by: Jani Nikula <[email protected]> > > So be it. I hope I don't regret letting gcc rule over us!
I figured we have git revert for regrets. ;) BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
