From 3ffe07e16cbba3f690198801696ec4251700903a Mon Sep 17 00:00:00 2001 From: Jakub Jelinek Date: Fri, 21 Jan 2011 16:15:40 +0100 Subject: [PATCH] re PR tree-optimization/47355 (ICE: verify_ssa failed with -O2 -fipa-cp-clone) PR tree-optimization/47355 * tree-eh.c (cleanup_empty_eh_merge_phis): Give up if NOP has non-debug uses beyond PHIs in new_bb. * g++.dg/opt/pr47355.C: New test. From-SVN: r169094 --- gcc/ChangeLog | 6 +++++ gcc/testsuite/ChangeLog | 5 ++++ gcc/testsuite/g++.dg/opt/pr47355.C | 39 ++++++++++++++++++++++++++++++ gcc/tree-eh.c | 14 +++++++++++ 4 files changed, 64 insertions(+) create mode 100644 gcc/testsuite/g++.dg/opt/pr47355.C diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 36ad66e6e88..09a73394d35 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2011-01-21 Jakub Jelinek + + PR tree-optimization/47355 + * tree-eh.c (cleanup_empty_eh_merge_phis): Give up if + NOP has non-debug uses beyond PHIs in new_bb. + 2011-01-21 Alexandre Oliva PR debug/47106 diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 61b3d23ab1c..8cef7033423 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2011-01-21 Jakub Jelinek + + PR tree-optimization/47355 + * g++.dg/opt/pr47355.C: New test. + 2011-01-21 Richard Guenther * gcc.dg/tree-ssa/pr47392.c: Make test non-static instead of diff --git a/gcc/testsuite/g++.dg/opt/pr47355.C b/gcc/testsuite/g++.dg/opt/pr47355.C new file mode 100644 index 00000000000..4fdbd9f5926 --- /dev/null +++ b/gcc/testsuite/g++.dg/opt/pr47355.C @@ -0,0 +1,39 @@ +// PR tree-optimization/47355 +// { dg-do compile } +// { dg-options "-O -fipa-cp -fipa-cp-clone" } + +struct T +{ + T (); + void *p; + ~T (); +}; + +void foo (T *i); + +T *bar (); +void baz (T *); + +struct V +{ + long q; + T *r; + ~V () + { + while (q) + { + foo (r); + ++r; + --q; + } + baz (r); + } +}; + +void +foo () +{ + V v; + T t; + v.r = bar (); +} diff --git a/gcc/tree-eh.c b/gcc/tree-eh.c index 65d0ff2f9fd..f189b9b6a96 100644 --- a/gcc/tree-eh.c +++ b/gcc/tree-eh.c @@ -3552,6 +3552,20 @@ cleanup_empty_eh_merge_phis (basic_block new_bb, basic_block old_bb, /* If we did find the corresponding PHI, copy those inputs. */ if (ophi) { + /* If NOP is used somewhere else beyond phis in new_bb, give up. */ + if (!has_single_use (nop)) + { + imm_use_iterator imm_iter; + use_operand_p use_p; + + FOR_EACH_IMM_USE_FAST (use_p, imm_iter, nop) + { + if (!gimple_debug_bind_p (USE_STMT (use_p)) + && (gimple_code (USE_STMT (use_p)) != GIMPLE_PHI + || gimple_bb (USE_STMT (use_p)) != new_bb)) + goto fail; + } + } bitmap_set_bit (ophi_handled, SSA_NAME_VERSION (nop)); FOR_EACH_EDGE (e, ei, old_bb->preds) {