aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Guenther <rguenther@suse.de>2008-09-16 12:31:51 +0000
committerRichard Biener <rguenth@gcc.gnu.org>2008-09-16 12:31:51 +0000
commit9cb57027a39541712b049df895beda264d73d5ca (patch)
treeb4ae6616a618305a5afad4b78279d4779209210a
parent3a5c03fba05f67c5cf8a089b5cca55a3885dc61a (diff)
downloadgcc-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
-rw-r--r--gcc/ChangeLog5
-rw-r--r--gcc/testsuite/ChangeLog5
-rw-r--r--gcc/testsuite/gcc.dg/tree-ssa/pr37508.c48
-rw-r--r--gcc/tree-vrp.c3
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)