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! -Chris _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
