[Bug c/81795] Stray "originally defined here" when using -Wc++-compat with #pragma GCC diagnostic push/pop

2017-10-19 Thread egallager at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81795

--- Comment #7 from Eric Gallager  ---
Compiling with an older GCC makes me wonder: could the fix for this bug also be
backported to the other open branches?

[Bug c/81795] Stray "originally defined here" when using -Wc++-compat with #pragma GCC diagnostic push/pop

2017-09-13 Thread aldyh at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81795

--- Comment #6 from Aldy Hernandez  ---
Author: aldyh
Date: Wed Sep 13 16:54:20 2017
New Revision: 252410

URL: https://gcc.gnu.org/viewcvs?rev=252410=gcc=rev
Log:
PR c/81795
* c-decl.c (pushtag): Only print inform if the warning was printed.
(grokdeclarator): Likewise.

* gcc.dg/pr81795.c: New test.

Added:
branches/range-gen2/gcc/testsuite/gcc.dg/pr81795.c
Modified:
branches/range-gen2/gcc/c/ChangeLog
branches/range-gen2/gcc/c/c-decl.c
branches/range-gen2/gcc/testsuite/ChangeLog

[Bug c/81795] Stray "originally defined here" when using -Wc++-compat with #pragma GCC diagnostic push/pop

2017-08-11 Thread egallager at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81795

--- Comment #5 from Eric Gallager  ---
(In reply to Marek Polacek from comment #4)
> Donezo.

Thanks for the quick fix!

[Bug c/81795] Stray "originally defined here" when using -Wc++-compat with #pragma GCC diagnostic push/pop

2017-08-11 Thread mpolacek at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81795

Marek Polacek  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Marek Polacek  ---
Donezo.

[Bug c/81795] Stray "originally defined here" when using -Wc++-compat with #pragma GCC diagnostic push/pop

2017-08-11 Thread mpolacek at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81795

--- Comment #3 from Marek Polacek  ---
Author: mpolacek
Date: Fri Aug 11 18:02:18 2017
New Revision: 251056

URL: https://gcc.gnu.org/viewcvs?rev=251056=gcc=rev
Log:
PR c/81795
* c-decl.c (pushtag): Only print inform if the warning was printed.
(grokdeclarator): Likewise.

* gcc.dg/pr81795.c: New test.

Added:
trunk/gcc/testsuite/gcc.dg/pr81795.c
Modified:
trunk/gcc/c/ChangeLog
trunk/gcc/c/c-decl.c
trunk/gcc/testsuite/ChangeLog

[Bug c/81795] Stray "originally defined here" when using -Wc++-compat with #pragma GCC diagnostic push/pop

2017-08-10 Thread mpolacek at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81795

Marek Polacek  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |mpolacek at gcc dot 
gnu.org

[Bug c/81795] Stray "originally defined here" when using -Wc++-compat with #pragma GCC diagnostic push/pop

2017-08-10 Thread egallager at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81795

--- Comment #2 from Eric Gallager  ---
(In reply to Marek Polacek from comment #1)
> You're right, do you want to send a patch or should I handle it?

It'd probably be better if you handled it; my hard drive that has the ssh keys
that let me commit is currently unresponsive and needs to be repaired, so even
if I sent the patch and got it approved, someone else would still need to
commit it for me.

[Bug c/81795] Stray "originally defined here" when using -Wc++-compat with #pragma GCC diagnostic push/pop

2017-08-10 Thread mpolacek at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81795

Marek Polacek  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2017-08-10
 CC||mpolacek at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #1 from Marek Polacek  ---
You're right, do you want to send a patch or should I handle it?