diff options
| author | Richard Guenther <rguenther@suse.de> | 2008-09-16 12:31:51 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2008-09-16 12:31:51 +0000 |
| commit | 9cb57027a39541712b049df895beda264d73d5ca (patch) | |
| tree | b4ae6616a618305a5afad4b78279d4779209210a /gcc | |
| parent | 3a5c03fba05f67c5cf8a089b5cca55a3885dc61a (diff) | |
| download | gcc-9cb57027a39541712b049df895beda264d73d5ca.zip gcc-9cb57027a39541712b049df895beda264d73d5ca.tar.gz gcc-9cb57027a39541712b049df895beda264d73d5ca.tar.bz2 | |
re PR tree-optimization/37508 (ICE in in simplify_truth_ops_using_ranges, at tree-vrp.c:6334)
2008-09-16 Richard Guenther <rguenther@suse.de>
PR tree-optimization/37508
* tree-vrp.c (simplify_truth_ops_using_ranges): Also allow -1.
* gcc.dg/tree-ssa/pr37508.c: New testcase.
From-SVN: r140387
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/tree-ssa/pr37508.c | 48 | ||||
| -rw-r--r-- | gcc/tree-vrp.c | 3 |
4 files changed, 60 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index e46d362..b413f20 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2008-09-16 Richard Guenther <rguenther@suse.de> + + PR tree-optimization/37508 + * tree-vrp.c (simplify_truth_ops_using_ranges): Also allow -1. + 2008-09-15 Kaz Kojima <kkojima@gcc.gnu.org> * config/sh/sh.md (movsf_ie): Fix length for TARGET_SH2A. diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 0288620..32c4435 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2008-09-16 Richard Guenther <rguenther@suse.de> + + PR tree-optimization/37508 + * gcc.dg/tree-ssa/pr37508.c: New testcase. + 2008-09-15 Aldy Hernandez <aldyh@redhat.com> * g++.old-deja/g++.brendan/crash16.C: Function name is the correct diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr37508.c b/gcc/testsuite/gcc.dg/tree-ssa/pr37508.c new file mode 100644 index 0000000..f31c594 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr37508.c @@ -0,0 +1,48 @@ +/* { dg-do compile } */ +/* { dg-options "-O2 -fdump-tree-vrp1" } */ + +struct foo1 { + int i:1; +}; +struct foo2 { + unsigned i:1; +}; + +int test1 (struct foo1 *x) +{ + if (x->i == 0) + return 1; + else if (x->i == -1) + return 1; + return 0; +} + +int test2 (struct foo2 *x) +{ + if (x->i == 0) + return 1; + else if (x->i == -1) + return 1; + return 0; +} + +int test3 (struct foo1 *x) +{ + if (x->i == 0) + return 1; + else if (x->i == 1) + return 1; + return 0; +} + +int test4 (struct foo2 *x) +{ + if (x->i == 0) + return 1; + else if (x->i == 1) + return 1; + return 0; +} + +/* { dg-final { scan-tree-dump-times "Folding" 4 "vrp1" { xfail *-*-* } } } */ +/* { dg-final { cleanup-tree-dump "vrp1" } } */ diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index 31d7f10..fbcda55 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -6331,7 +6331,8 @@ simplify_truth_ops_using_ranges (gimple_stmt_iterator *gsi, gimple stmt) /* Exclude anything that should have been already folded. */ gcc_assert (rhs_code == EQ_EXPR || rhs_code == NE_EXPR || rhs_code == TRUTH_XOR_EXPR); - gcc_assert (integer_zerop (op1) || integer_onep (op1)); + gcc_assert (integer_zerop (op1) || integer_onep (op1) + || integer_all_onesp (op1)); /* Limit the number of cases we have to consider. */ if (rhs_code == EQ_EXPR) |