On Fri, 2016-01-29 at 20:50 +0100, Jakub Jelinek wrote: > Hi! > > This patch reverts one tiny change from r228049 changes (which hasn't > been > mentioned in the ChangeLog or patch description). We definitely need > to > revisit this for GCC 7, but stage4 is probably not the right time for > that, > and the patch fixes e.g. tons of warnings (or with -Werror errors on > including pretty much all glib2 headers). > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > 2016-01-29 Jakub Jelinek <ja...@redhat.com> > > PR preprocessor/69543 > PR c/69558 > * c-pragma.c (handle_pragma_diagnostic): Pass input_location > instead of loc to control_warning_option. > > * gcc.dg/pr69543.c: New test. > * gcc.dg/pr69558.c: New test.
This touches c-family; shouldn't the new tests be in c-c++-common, rather than gcc.dg? (presumably we need to ensure that the glib2 headers are sane from C++ also) I've been attempting to fix these by fixing linemap_compare_locations, but I don't have that approach working, so fwiw I don't object to this patch.
