On Mon, 2017-10-30 at 17:29 +0000, Chris Wilson wrote: > To quote kbuild/makefiles.txt: > > cc-disable-warning checks if gcc supports a given warning and returns > the commandline switch to disable it. This special function is needed, > because gcc 4.4 and later accept any unknown -Wno-* option and only > warn about it if there is another warning in the source file. > > This is exactly what we were trying to achieve with cc-option -Wno-foo and > failed miserably. > > Reported-by: [email protected] > Fixes: 39bf4de89ff7 ("drm/i915: Add -Wall -Wextra to our build, set warnings > to full") > Signed-off-by: Chris Wilson <[email protected]> > Cc: Joonas Lahtinen <[email protected]> > Cc: Jani Nikula <[email protected]>
Poor person who greps for "-Wno-", should we add comment "# -Wno-whatever" at the end of each line? Reviewed-by: Joonas Lahtinen <[email protected]> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
