[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-04-15 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 --- Comment #19 from Jakub Jelinek --- Author: jakub Date: Fri Apr 15 12:24:18 2016 New Revision: 235020 URL: https://gcc.gnu.org/viewcvs?rev=235020=gcc=rev Log: PR c/70436 c/ * c-parser.c (c_parser_pragma): Add IF_P argument,

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-04-15 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 --- Comment #18 from Jakub Jelinek --- Created attachment 38279 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38279=edit gcc6-pr70436-omp.patch Untested fix for OpenMP/OpenACC/Cilk+/#pragma GCC ivdep, both C and C++.

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-04-13 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 Marek Polacek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-04-13 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 --- Comment #16 from Marek Polacek --- Author: mpolacek Date: Wed Apr 13 16:00:52 2016 New Revision: 234949 URL: https://gcc.gnu.org/viewcvs?rev=234949=gcc=rev Log: PR c/70436 * c-parser.c (c_parser_statement_after_labels): Add

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-04-08 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 Marek Polacek changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-04-06 Thread ppalka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 --- Comment #14 from Patrick Palka --- Author: ppalka Date: Wed Apr 6 23:10:14 2016 New Revision: 234802 URL: https://gcc.gnu.org/viewcvs?rev=234802=gcc=rev Log: Fix C++ side of PR c/70436 (missing -Wparentheses warnings) gcc/cp/ChangeLog:

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-04-06 Thread ppalka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 --- Comment #13 from Patrick Palka --- Author: ppalka Date: Wed Apr 6 23:07:21 2016 New Revision: 234801 URL: https://gcc.gnu.org/viewcvs?rev=234801=gcc=rev Log: Fix new -Wparentheses warnings encountered during bootstrap gcc/ChangeLog:

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-04-05 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 Richard Biener changed: What|Removed |Added Keywords||diagnostic Priority|P3

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-03-31 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 --- Comment #12 from Jakub Jelinek --- They should be. if (x) #pragma omp for for (...) if (y) ... else ... and #pragma omp simd and #pragma omp taskloop too. For C++, perhaps we could just pass around if_p argument to a few more

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-03-31 Thread ppalka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 --- Comment #11 from Patrick Palka --- Should non-standard constructs be considered in this PR? I noticed that we also don't warn on if (a) #pragma GCC ivdep while (1) if (b) bar (); else baz (); and if (a)

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-03-29 Thread joseph at codesourcery dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 --- Comment #10 from joseph at codesourcery dot com --- I'd consider it a bug that the existing -Wparentheses warning for ambiguous "else" doesn't warn in this case. As with the existing warning, that warning is only appropriate when there is

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-03-29 Thread bernds at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 --- Comment #9 from Bernd Schmidt --- I suspect what broke it was git revision 0375a27521885.

[Bug c/70436] [4.9/5/6 Regression] -Wparentheses missing ambiguous else warning

2016-03-29 Thread bernds at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70436 Bernd Schmidt changed: What|Removed |Added CC||bernds at gcc dot gnu.org