* Marek Polacek: >> +void >> +implicit_function_declaration (void) >> +{ >> + f1 (); /* { dg-warning "'f1' \\\[-Wimplicit-function-declaration\\\]" } */ >> +} >> + >> +extern implicit_int_1; /* { dg-warning "'implicit_int_1' >> \\\[-Wimplicit-int\\\]" } */ > > Oy, these \ tend to get unwieldy. You could probably just say > { dg-warning {-Wimplicit-int} }
I wanted to have some more context for the other files that might get the explict line number wrong. >> + return incompatible_pointer_types; /* { dg-error "returning 'int \\\* >> \\\(\\\*\\\)\\\(int\\\)' from a function with incompatible return type 'int >> \\\*' \\\[-Wincompatible-pointer-types\\\]" } */ > > And here maybe > { dg-error {returning 'int \* \(\*\)\(int\)' from a function with > incompatible return type 'int \*' \[-Wincompatible-pointer-types\]} } > could work the same. But you don't have to go and change it; I don't > want to make more work for you. Oh, I'm not too familiar with Tcl. I haven't seen single quotes being used elsewhere. I prefer not to change it unless there is another reason to rework all this. So … let's wait? >> --- /dev/null >> +++ b/gcc/testsuite/gcc.dg/permerror-system.c >> @@ -0,0 +1,9 @@ >> +/* { dg-options "-isystem ${srcdir}" } */ >> + >> +/* Test that permerrors appear in system headers. */ >> + >> +/* The dg-* directives in the header file are ignored. No warnings are >> + expected. */ >> +#include <gcc.dg/permerror-default.c> > > Why not just #include "permerror-default.c"? I wanted to make sure that the file is found through the -isystem path. Thanks, Florian