[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2023-07-29 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 Andrew Pinski changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2020-12-10 Thread nico at josuttis dot de via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 --- Comment #13 from Nicolai Josuttis --- Oh, sorry, your are right, the example indeed works. BUT: I used in fact a slightly different example (sorry, didn't expect that there is a difference): int main() { int i = 0; int j = i++ << i++;

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2020-12-07 Thread mpolacek at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 Marek Polacek changed: What|Removed |Added CC||mpolacek at gcc dot gnu.org --- Comment

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2020-12-03 Thread nico at josuttis dot de via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 Nicolai Josuttis changed: What|Removed |Added CC||nico at josuttis dot de --- Comment

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-27 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 --- Comment #10 from Jakub Jelinek --- Partially fixed, needs more work, so keeping this open.

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-27 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 --- Comment #9 from Jakub Jelinek --- Author: jakub Date: Tue Aug 27 12:37:30 2019 New Revision: 274952 URL: https://gcc.gnu.org/viewcvs?rev=274952=gcc=rev Log: PR c++/91415 * c-common.c (verify_tree): For LSHIFT_EXPR,

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-12 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 --- Comment #8 from Jakub Jelinek --- Oh, and need to look also at CALL_EXPR.

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-12 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 --- Comment #7 from Jakub Jelinek --- On top of the above, I've tried: --- gcc/c-family/c-common.c.jj 2019-08-12 09:45:54.463491950 +0200 +++ gcc/c-family/c-common.c 2019-08-12 12:01:32.783135654 +0200 @@ -1933,16 +1934,19 @@ verify_tree

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-12 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 --- Comment #6 from Jakub Jelinek --- Created attachment 46702 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46702=edit gcc10-pr91415-1.patch Let's handle this incrementally, from the easiest cases to the hardest.

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-10 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 --- Comment #5 from Marek Polacek --- I think this is P0145R3 Refining Expression Evaluation Order for Idiomatic C++ http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2016/p0145r3.pdf which says 4. A SOLUTION In summary, the following

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-10 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 Marek Polacek changed: What|Removed |Added Keywords||diagnostic

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-10 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 --- Comment #4 from Jakub Jelinek --- Testcase also with PMF: struct S { int a[10]; void bar (); void baz (); }; typedef void (S::*pmf) (); void foo (int i, int x[10][10], int y[10], struct S z[10], struct S *w[10], pmf u[10]) { int b =

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-10 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 --- Comment #3 from Jakub Jelinek --- --- gcc/c-family/c-common.c.jj 2019-07-30 08:27:49.987555303 +0200 +++ gcc/c-family/c-common.c 2019-08-10 18:13:20.821949299 +0200 @@ -1889,6 +1889,7 @@ verify_tree (tree x, struct tlist **pbef

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-10 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org,

[Bug c++/91415] Invalid warning for C++17 sequencing of shift operator E1<

2019-08-10 Thread maxim.yegorushkin at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91415 --- Comment #1 from Maxim Egorushkin --- gcc-9.1 produces the same warning.