[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Jakub Jelinek changed: What|Removed |Added Target Milestone|8.5 |--- CC||jakub at gcc dot gnu.org
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Jakub Jelinek changed: What|Removed |Added Target Milestone|8.4 |8.5 --- Comment #14 from Jakub Jelinek --- GCC 8.4.0 has been released, adjusting target milestone.
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Jakub Jelinek changed: What|Removed |Added Target Milestone|8.3 |8.4 --- Comment #13 from Jakub Jelinek --- GCC 8.3 has been released.
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Jakub Jelinek changed: What|Removed |Added Target Milestone|8.2 |8.3 --- Comment #12 from Jakub Jelinek --- GCC 8.2 has been released.
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Jakub Jelinek changed: What|Removed |Added Target Milestone|8.0 |8.2 --- Comment #11 from Jakub Jelinek --- GCC 8.1 has been released.
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Martin Liška changed: What|Removed |Added Status|ASSIGNED|NEW Assignee|marxin at gcc dot gnu.org |unassigned at gcc dot gnu.org --- Comment #10 from Martin Liška --- I'm surrendering here..
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Eric Gallager changed: What|Removed |Added Keywords||patch URL||https://gcc.gnu.org/ml/gcc- ||patches/2018-04/msg00063.ht ||ml CC||egallager at gcc dot gnu.org --- Comment #9 from Eric Gallager --- Patch discussion starts here: https://gcc.gnu.org/ml/gcc-patches/2018-04/msg00063.html
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Martin Liška changed: What|Removed |Added Target Milestone|--- |8.0
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Martin Liška changed: What|Removed |Added Status|NEW |ASSIGNED CC||marxin at gcc dot gnu.org Assignee|unassigned at gcc dot gnu.org |marxin at gcc dot gnu.org
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Eric Gallager changed: What|Removed |Added CC||Patrick.Schluter at ec dot europa. ||eu --- Comment #8 from Eric Gallager --- *** Bug 84179 has been marked as a duplicate of this bug. ***
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 --- Comment #7 from Mark Wielaard --- The workaround is to use gcc -C --save-temps ... to pass-through all comments to the temp files. Maybe -C should be the default with --save-temps?
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497
Mark Wielaard changed:
What|Removed |Added
CC||mark at gcc dot gnu.org
--- Comment #6 from Mark Wielaard ---
Confirmed, this confused me a bit.
Take t.c:
int main (int argc, char **argv)
{
int a;
switch (argc)
{
case 1:
a = 1;
break;
case 2:
a = 2;
/* FALLTHROUGH */
case 3:
a = 3;
break;
}
return a;
}
$ gcc -Werror -Wimplicit-fallthrough t.c
is fine, but...
$ gcc --save-temps -Werror -Wimplicit-fallthrough t.c
t.c: In function ‘main’:
t.c:10:9: error: this statement may fall through
[-Werror=implicit-fallthrough=]
a = 2;
~~^~~
t.c:12:5: note: here
case 3:
^~~~
cc1: all warnings being treated as errors
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org --- Comment #5 from Manuel López-Ibáñez --- (In reply to Richard Biener from comment #2) > Confirmed. I'm also not sure we can do much about this... (inject a #pragma > diagnostic into preprocessed output?) Or don't drop those "special" comments?
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Eric Gallager changed: What|Removed |Added CC||gjl at gcc dot gnu.org --- Comment #4 from Eric Gallager --- *** Bug 81582 has been marked as a duplicate of this bug. ***
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 --- Comment #3 from Alexandre Oliva --- One aspect to the problem is the extra warnings you get. As more and more projects adopt -Werror by default, it means you get a different error with -save-temps. That's bad for users and inconvenient for ourselves. Now the real kicker is that -save-temps is the standard, recommended way for users to report bugs. As preprocessing now discards information that feeds into compiler decisions, the odds of some other divergence that is more likely to affect small details in the compilation to the point of no longer triggering hard-to-hit bugs grows. That would is a very unfortunate outcome for a problem that has an easy-ish solution described in the bug report description.
[Bug preprocessor/78497] compiling with -save-temps adds -Wimplicit-fallthrough warnings
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78497 Richard Biener changed: What|Removed |Added Keywords||diagnostic Status|UNCONFIRMED |NEW Last reconfirmed||2016-11-24 Component|regression |preprocessor Ever confirmed|0 |1 --- Comment #2 from Richard Biener --- Confirmed. I'm also not sure we can do much about this... (inject a #pragma diagnostic into preprocessed output?)
