diff options
| author | Richard Guenther <rguenther@suse.de> | 2008-04-06 21:20:49 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2008-04-06 21:20:49 +0000 |
| commit | 3633df5a3dba55008be0049598e0377dfa3809ac (patch) | |
| tree | e96c1e164c8e0a70789c96a57a9bd3fd508a3528 /gcc | |
| parent | 5dc77808914b3924b156c1960636d7c320cdd4c5 (diff) | |
| download | gcc-3633df5a3dba55008be0049598e0377dfa3809ac.zip gcc-3633df5a3dba55008be0049598e0377dfa3809ac.tar.gz gcc-3633df5a3dba55008be0049598e0377dfa3809ac.tar.bz2 | |
re PR middle-end/35400 (-Wtype-limits -O2 causes ICE tree check: expected ssa_name, have addr_expr in get_value_range, at tree-vrp.c:469)
2008-04-06 Richard Guenther <rguenther@suse.de>
PR tree-optimization/35400
* tree-vrp.c (vrp_evaluate_conditional): Only query value-range
information from SSA_NAMEs.
* gcc.dg/torture/pr35400.c: New testcase.
* g++.dg/torture/pr35400.C: Likewise.
From-SVN: r133967
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/testsuite/g++.dg/torture/pr35400.C | 13 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/torture/pr35400.c | 22 | ||||
| -rw-r--r-- | gcc/tree-vrp.c | 3 |
5 files changed, 49 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index da2f5a7..921982e 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2008-04-06 Richard Guenther <rguenther@suse.de> + + PR tree-optimization/35400 + * tree-vrp.c (vrp_evaluate_conditional): Only query value-range + information from SSA_NAMEs. + 2008-04-06 Anatoly Sokolov <aesok@post.ru> * config/avr/avr.h (avr_mega_p): Remove declaration. diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index b5b2155..beac0eb 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2008-04-06 Richard Guenther <rguenther@suse.de> + + PR tree-optimization/35400 + * gcc.dg/torture/pr35400.c: New testcase. + * g++.dg/torture/pr35400.C: Likewise. + 2008-04-06 Paul Thomas <pault@gcc.gnu.org> PR fortran/35780 diff --git a/gcc/testsuite/g++.dg/torture/pr35400.C b/gcc/testsuite/g++.dg/torture/pr35400.C new file mode 100644 index 0000000..021135d --- /dev/null +++ b/gcc/testsuite/g++.dg/torture/pr35400.C @@ -0,0 +1,13 @@ +/* { dg-do compile } */ +/* { dg-options "-Wtype-limits" } */ + +struct A +{ + A(); + ~A(); +}; + +void foo() +{ + A x[1]; +} diff --git a/gcc/testsuite/gcc.dg/torture/pr35400.c b/gcc/testsuite/gcc.dg/torture/pr35400.c new file mode 100644 index 0000000..285dc72 --- /dev/null +++ b/gcc/testsuite/gcc.dg/torture/pr35400.c @@ -0,0 +1,22 @@ +/* { dg-do compile } */ +/* { dg-options "-Wtype-limits" } */ + +struct A +{ + struct A *p; +}; + +int foo(const struct A *q) +{ + return q->p == q; +} + +void bar(int); + +void baz() +{ + struct A a; + + while (foo(&a)) + bar(foo(&a)); +} diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index 9741262..273b5fd 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -5385,7 +5385,8 @@ vrp_evaluate_conditional (tree cond, tree stmt) if (warn_type_limits && ret - && TREE_CODE_CLASS (TREE_CODE (cond)) == tcc_comparison) + && TREE_CODE_CLASS (TREE_CODE (cond)) == tcc_comparison + && TREE_CODE (TREE_OPERAND (cond, 0)) == SSA_NAME) { /* If the comparison is being folded and the operand on the LHS is being compared against a constant value that is outside of |