diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2010-12-16 22:44:02 +0100 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2010-12-16 22:44:02 +0100 |
| commit | bcdae749d017a08b7298f86dd732b91edb84c2bd (patch) | |
| tree | a6b941ced8b02130c5e8c8bcf0b55996d0d50ca7 | |
| parent | 94406344b56752b35b729e97a85ce2a922ad6338 (diff) | |
| download | gcc-bcdae749d017a08b7298f86dd732b91edb84c2bd.zip gcc-bcdae749d017a08b7298f86dd732b91edb84c2bd.tar.gz gcc-bcdae749d017a08b7298f86dd732b91edb84c2bd.tar.bz2 | |
re PR tree-optimization/43655 (-ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test)
PR tree-optimization/43655
* tree-ssa-ter.c (is_replaceable_p): Don't use
gimple_references_memory_p for -O0, instead check for load
by looking at rhs.
* g++.dg/opt/pr43655.C: New test.
From-SVN: r167955
| -rw-r--r-- | gcc/ChangeLog | 7 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/g++.dg/opt/pr43655.C | 34 | ||||
| -rw-r--r-- | gcc/tree-ssa-ter.c | 4 |
4 files changed, 49 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 287d21e..de5aae5 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2010-12-16 Jakub Jelinek <jakub@redhat.com> + + PR tree-optimization/43655 + * tree-ssa-ter.c (is_replaceable_p): Don't use + gimple_references_memory_p for -O0, instead check for load + by looking at rhs. + 2010-12-16 Sebastian Pop <sebastian.pop@amd.com> PR tree-optimization/46404 diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 8cfb516..b5c1508 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-12-16 Jakub Jelinek <jakub@redhat.com> + + PR tree-optimization/43655 + * g++.dg/opt/pr43655.C: New test. + 2010-12-16 Sebastian Pop <sebastian.pop@amd.com> PR tree-optimization/46404 diff --git a/gcc/testsuite/g++.dg/opt/pr43655.C b/gcc/testsuite/g++.dg/opt/pr43655.C new file mode 100644 index 0000000..f7e370b --- /dev/null +++ b/gcc/testsuite/g++.dg/opt/pr43655.C @@ -0,0 +1,34 @@ +// PR tree-optimization/43655 +// { dg-do run } +// { dg-options "-O0 -ftree-ter" } + +extern "C" void abort (); + +struct C +{ + C (int i) : val(i) { } + C (const C& c) : val(c.val) { } + ~C (void) { val = 999; } + C& operator = (const C& c) { val = c.val; return *this; } + C& inc (int i) { val += i; return *this; } + int val; +}; + +C +f () +{ + return C (3); +} + +C +f (int i) +{ + return f ().inc (i); +} + +int +main () +{ + if (f (2).val != 5) + abort (); +} diff --git a/gcc/tree-ssa-ter.c b/gcc/tree-ssa-ter.c index 7bd7669..47954cf 100644 --- a/gcc/tree-ssa-ter.c +++ b/gcc/tree-ssa-ter.c @@ -416,7 +416,9 @@ is_replaceable_p (gimple stmt) return false; /* Without alias info we can't move around loads. */ - if (gimple_references_memory_p (stmt) && !optimize) + if (!optimize + && gimple_assign_single_p (stmt) + && !is_gimple_val (gimple_assign_rhs1 (stmt))) return false; /* Float expressions must go through memory if float-store is on. */ |