From: rguenth Date: Tue, 18 Sep 2007 08:46:23 +0000 (+0000) Subject: 2007-09-18 Richard Guenther X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=5dbdbadce9abd166b3e176fb9752352e3bc25027;p=thirdparty%2Fgcc.git 2007-09-18 Richard Guenther PR tree-optimization/33340 * tree-ssa-sccvn.c (set_ssa_val_to): Do not set values to SSA_NAMEs that occur in abnormal PHI nodes. * g++.dg/torture/pr33340.C: New testcase. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@128571 138bc75d-0d04-0410-961f-82ee72b054a4 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index d9f37e82e5ad..f8a9aff6c15c 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2007-09-18 Richard Guenther + + PR tree-optimization/33340 + * tree-ssa-sccvn.c (set_ssa_val_to): Do not set values to + SSA_NAMEs that occur in abnormal PHI nodes. + 2007-09-17 Kaveh R. Ghazi * tree-cfg.c (is_ctrl_altering_stmt, tree_block_ends_with_call_p): diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index a07270af87de..3437101f9355 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2007-09-18 Richard Guenther + + PR tree-optimization/33340 + * g++.dg/torture/pr33340.C: New testcase. + 2007-09-18 Tobias Burnus PR fortran/33231 diff --git a/gcc/testsuite/g++.dg/torture/pr33340.C b/gcc/testsuite/g++.dg/torture/pr33340.C new file mode 100644 index 000000000000..bac882156b55 --- /dev/null +++ b/gcc/testsuite/g++.dg/torture/pr33340.C @@ -0,0 +1,27 @@ +void* operator new(__SIZE_TYPE__, void* __p) { } + +struct auto_ptr { + int* p; + ~auto_ptr() { delete p; } +}; + +typedef void* T; +struct vector { + void push_back(const T& __x) { + ::new(0) T(__x); + insert(__x); + } + void insert(const T& __x); +} v; + +void g(); +void f() { + auto_ptr ap; + if (ap.p) { + ap.p = new int(); + } + g(); + int* tmp = ap.p; + ap.p = 0; + v.push_back(tmp); +} diff --git a/gcc/tree-ssa-sccvn.c b/gcc/tree-ssa-sccvn.c index bca0e8473aeb..cddd2d1df58e 100644 --- a/gcc/tree-ssa-sccvn.c +++ b/gcc/tree-ssa-sccvn.c @@ -1022,6 +1022,11 @@ set_ssa_val_to (tree from, tree to) { tree currval; + if (from != to + && TREE_CODE (to) == SSA_NAME + && SSA_NAME_OCCURS_IN_ABNORMAL_PHI (to)) + to = from; + /* The only thing we allow as value numbers are VN_TOP, ssa_names and invariants. So assert that here. */ gcc_assert (to != NULL_TREE