diff options
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/torture/pr25481.c | 18 | ||||
| -rw-r--r-- | gcc/tree-ssa-structalias.c | 10 |
4 files changed, 36 insertions, 3 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 98468169d1c..6c9d1b50971 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2005-12-18 Richard Guenther <rguenther@suse.de> + + PR tree-optimization/25481 + * tree-ssa-structalias.c (handle_ptr_arith): Handle + accesses we don't have a varinfo for. + 2005-12-17 Jon Grimm <jgrimm2@us.ibm.com> Janis Johnson <janis187@us.ibm.com> Ben Elliston <bje@au.ibm.com> diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 41749611272..c12b7885c07 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2005-12-18 Richard Guenther <rguenther@suse.de> + + PR tree-optimization/25481 + * gcc.dg/torture/pr25481.c: New testcase. + 2005-12-18 Ulrich Weigand <uweigand@de.ibm.com> PR rtl-optimization/21041 diff --git a/gcc/testsuite/gcc.dg/torture/pr25481.c b/gcc/testsuite/gcc.dg/torture/pr25481.c new file mode 100644 index 00000000000..3072e5ecb76 --- /dev/null +++ b/gcc/testsuite/gcc.dg/torture/pr25481.c @@ -0,0 +1,18 @@ +/* { dg-do compile } */ + +struct s { + int *blah; +}; + +static struct s array[] = { + { 0 } +}; + +void +foo (struct s *p) +{ + unsigned int n = 1; + struct s *q = &array[n]; + while (p < q) + p++; +} diff --git a/gcc/tree-ssa-structalias.c b/gcc/tree-ssa-structalias.c index cf307593b3c..b4251d6f1e3 100644 --- a/gcc/tree-ssa-structalias.c +++ b/gcc/tree-ssa-structalias.c @@ -3184,9 +3184,13 @@ handle_ptr_arith (VEC (ce_s, heap) *lhsc, tree expr) if (c2->type == ADDRESSOF && rhsoffset != 0) { varinfo_t temp = get_varinfo (c2->var); - - gcc_assert (first_vi_for_offset (temp, rhsoffset) != NULL); - c2->var = first_vi_for_offset (temp, rhsoffset)->id; + + /* An access one after the end of an array is valid, + so simply punt on accesses we cannot resolve. */ + temp = first_vi_for_offset (temp, rhsoffset); + if (temp == NULL) + continue; + c2->var = temp->id; c2->offset = 0; } else |