[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #29 from David Malcolm --- Author: dmalcolm Date: Tue Feb 23 17:44:28 2016 New Revision: 233638 URL: https://gcc.gnu.org/viewcvs?rev=233638&root=gcc&view=rev Log: PR preprocessor/69126: avoid comparing ad-hoc and non-ad-hoc locations

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #28 from David Malcolm --- Author: dmalcolm Date: Tue Feb 23 17:39:16 2016 New Revision: 233637 URL: https://gcc.gnu.org/viewcvs?rev=233637&root=gcc&view=rev Log: Add test coverage for _Pragma (PR preprocessor 69126, 69543, 69558) W

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-22 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 Paolo Carlini changed: What|Removed |Added CC||paolo.carlini at oracle dot com --- Comm

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-20 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #26 from David Malcolm --- (In reply to David Malcolm from comment #25) [...] > I have a patch that seems to work for this test case; am testing it more > thoroughly now. Candidate patch posted here: https://gcc.gnu.org/ml/gcc-patch

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-19 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 David Malcolm changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #25 from David Malc

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-01 Thread sbergman at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #24 from Stephan Bergmann --- (In reply to Manuel López-Ibáñez from comment #23) > That is weird. If you use "GCC diagnostic warning" instead of "ignored", you > should be able to see some changes in locations between -O0 and -O1 and

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-01 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #23 from Manuel López-Ibáñez --- (In reply to Stephan Bergmann from comment #18) > (Also, the problem > only happens with -O or higher.) That is weird. If you use "GCC diagnostic warning" instead of "ignored", you should be able to s

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-01 Thread sbergman at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #22 from Stephan Bergmann --- (In reply to Jakub Jelinek from comment #20) > Does the http://gcc.gnu.org/ml/gcc-patches/2016-01/msg02347.html workaround > help here in any way? No, unfortunately not. Doesn't change the behaviour in

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #21 from Jakub Jelinek --- In any case, we need much better testsuite coverage for the cases that the various projects use (glib2, LO, ...), so that we don't regress on those without knowing about it months later.

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #20 from Jakub Jelinek --- Does the http://gcc.gnu.org/ml/gcc-patches/2016-01/msg02347.html workaround help here in any way?

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 Jakub Jelinek changed: What|Removed |Added Status|RESOLVED|NEW Resolution|FIXED

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-02-01 Thread sbergman at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 Stephan Bergmann changed: What|Removed |Added CC||sbergman at redhat dot com --- Commen

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-01-27 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-01-27 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #16 from David Malcolm --- Author: dmalcolm Date: Wed Jan 27 18:57:51 2016 New Revision: 232893 URL: https://gcc.gnu.org/viewcvs?rev=232893&root=gcc&view=rev Log: libcpp: use better locations for _Pragma tokens (preprocessor/69126)

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-01-17 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #15 from Manuel López-Ibáñez --- (In reply to Paolo Carlini from comment #2) > the regression and I'm pretty sure would not introduce other wrt the status > pre-r226234, but I don't fully understand why that is necessary, there are W

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-01-17 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #14 from Manuel López-Ibáñez --- Related bug 52116

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-01-17 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org --- Commen

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-01-17 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 --- Comment #12 from David Malcolm --- Patch posted here: https://gcc.gnu.org/ml/gcc-patches/2016-01/msg01212.html

[Bug preprocessor/69126] [6 regression] _Pragma does not apply if part of a macro

2016-01-15 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69126 David Malcolm changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned a