diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2017-09-27 16:19:57 +0200 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2017-09-27 16:19:57 +0200 |
| commit | b55c7343f02718bca7a8d2eba9dc9e6e3905d9e8 (patch) | |
| tree | f1786b6bc23f822bbf3b6abba22114c8876894cf | |
| parent | 99124c31f9afaa8e713f3e1335d8e53c2ee61bc1 (diff) | |
| download | gcc-b55c7343f02718bca7a8d2eba9dc9e6e3905d9e8.zip gcc-b55c7343f02718bca7a8d2eba9dc9e6e3905d9e8.tar.gz gcc-b55c7343f02718bca7a8d2eba9dc9e6e3905d9e8.tar.bz2 | |
re PR c++/82159 (ICE: in assign_temp, at function.c:961)
PR c++/82159
* gimplify.c (gimplify_modify_expr): Don't optimize away zero sized
lhs from calls if the lhs has addressable type.
* g++.dg/opt/pr82159.C: New test.
From-SVN: r253230
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/gimplify.c | 7 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/g++.dg/opt/pr82159.C | 18 |
4 files changed, 35 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index d35c497..44783aa 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2017-09-27 Jakub Jelinek <jakub@redhat.com> + + PR c++/82159 + * gimplify.c (gimplify_modify_expr): Don't optimize away zero sized + lhs from calls if the lhs has addressable type. + 2017-09-27 Richard Biener <rguenther@suse.de> * graphite.h (scop::max_alias_set): New member. diff --git a/gcc/gimplify.c b/gcc/gimplify.c index 8b29a71..c3fd6ac 100644 --- a/gcc/gimplify.c +++ b/gcc/gimplify.c @@ -5479,7 +5479,12 @@ gimplify_modify_expr (tree *expr_p, gimple_seq *pre_p, gimple_seq *post_p, side as statements and throw away the assignment. Do this after gimplify_modify_expr_rhs so we handle TARGET_EXPRs of addressable types properly. */ - if (zero_sized_type (TREE_TYPE (*from_p)) && !want_value) + if (zero_sized_type (TREE_TYPE (*from_p)) + && !want_value + /* Don't do this for calls that return addressable types, expand_call + relies on those having a lhs. */ + && !(TREE_ADDRESSABLE (TREE_TYPE (*from_p)) + && TREE_CODE (*from_p) == CALL_EXPR)) { gimplify_stmt (from_p, pre_p); gimplify_stmt (to_p, pre_p); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index a61d510..39682ce 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2017-09-27 Jakub Jelinek <jakub@redhat.com> + + PR c++/82159 + * g++.dg/opt/pr82159.C: New test. + 2017-09-27 Andreas Krebbel <krebbel@linux.vnet.ibm.com> * gcc.dg/vect/pr65947-9.c: Use signed char explicitly. diff --git a/gcc/testsuite/g++.dg/opt/pr82159.C b/gcc/testsuite/g++.dg/opt/pr82159.C new file mode 100644 index 0000000..e39dbc3 --- /dev/null +++ b/gcc/testsuite/g++.dg/opt/pr82159.C @@ -0,0 +1,18 @@ +// PR c++/82159 +// { dg-do compile } +// { dg-options "" } + +template<int N> +struct S +{ + ~S () {} + template<int M> S<M> foo () { return S<M> (); } + unsigned char data[N]; +}; + +int +main () +{ + S<16> d; + S<0> t = d.foo<0> (); +} |