[Bug c++/79232] [7 Regression] error: invalid rhs for gimple memory store

2017-02-13 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79232 Jakub Jelinek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/79232] [7 Regression] error: invalid rhs for gimple memory store

2017-02-13 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79232 --- Comment #9 from Jakub Jelinek --- Author: jakub Date: Mon Feb 13 19:31:14 2017 New Revision: 245401 URL: https://gcc.gnu.org/viewcvs?rev=245401&root=gcc&view=rev Log: PR c++/79232 * typeck.c (cp_build_modify_expr): Handle pro

[Bug c++/79232] [7 Regression] error: invalid rhs for gimple memory store

2017-01-30 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79232 Jakub Jelinek changed: What|Removed |Added Attachment #40607|0 |1 is obsolete|

[Bug c++/79232] [7 Regression] error: invalid rhs for gimple memory store

2017-01-27 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79232 --- Comment #7 from Jakub Jelinek --- Created attachment 40607 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40607&action=edit gcc7-pr79232.patch This patch (totally untested) implements it and fixes the ICE with both the original reduced

[Bug c++/79232] [7 Regression] error: invalid rhs for gimple memory store

2017-01-27 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79232 --- Comment #6 from Jakub Jelinek --- Though the #c1 testcase actually tests something even different (the #c5 questions apply otherwise), because there is the [d], therefore it really should be evaluated in the order: side-effects in e, side-eff

[Bug c++/79232] [7 Regression] error: invalid rhs for gimple memory store

2017-01-27 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79232 --- Comment #5 from Jakub Jelinek --- On the C++, the question really is what C++17 wants with comma expression on the lhs used as lvalue and what should be done for C++14 and earlier. Shouldn't COMPOUND_EXPR be handled like before for -fstrong-

[Bug c++/79232] [7 Regression] error: invalid rhs for gimple memory store

2017-01-27 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79232 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org,

[Bug c++/79232] [7 Regression] error: invalid rhs for gimple memory store

2017-01-26 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79232 Martin Liška changed: What|Removed |Added CC||jason at gcc dot gnu.org,

[Bug c++/79232] [7 Regression] error: invalid rhs for gimple memory store

2017-01-26 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79232 Richard Biener changed: What|Removed |Added Priority|P3 |P1 Status|UNCONFIRMED