aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKazu Hirata <kazu@cs.umass.edu>2005-05-14 20:32:59 +0000
committerKazu Hirata <kazu@gcc.gnu.org>2005-05-14 20:32:59 +0000
commit5de2df7bd4a71a70ecbb3f6c514183864dc75c8a (patch)
treed87d80e20a98cc3925d2abf5c81ae16134950b55
parent92df92cddae0295ede176f971612090ef32dfbf9 (diff)
downloadgcc-5de2df7bd4a71a70ecbb3f6c514183864dc75c8a.zip
gcc-5de2df7bd4a71a70ecbb3f6c514183864dc75c8a.tar.gz
gcc-5de2df7bd4a71a70ecbb3f6c514183864dc75c8a.tar.bz2
re PR tree-optimization/21563 (A trivial VRP opportunity missed)
gcc/ PR tree-optimization/21563 * tree-vrp.c (compare_value): Return boolean_false_node when SSA_NAME in "if (SSA_NAME == CST)" is strictly smaller than or strictly larger than CST. testsuite/ PR tree-optimization/21563 * gcc.dg/tree-ssa/pr21563.c: New. From-SVN: r99705
-rw-r--r--gcc/ChangeLog5
-rw-r--r--gcc/testsuite/ChangeLog5
-rw-r--r--gcc/testsuite/gcc.dg/tree-ssa/pr21563.c17
-rw-r--r--gcc/tree-vrp.c3
4 files changed, 30 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 2338389..2a1eee8 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -3,6 +3,11 @@
* tree-optimize.c (init_tree_optimization_passes): Move
pass_cd_dce in place of the last pass_dce.
+ PR tree-optimization/21563
+ * tree-vrp.c (compare_value): Return boolean_false_node when
+ SSA_NAME in "if (SSA_NAME == CST)" is strictly smaller than or
+ strictly larger than CST.
+
2005-05-14 Nathan Sidwell <nathan@codesourcery.com>
Jan-Benedict Glaw <jbglaw@lug-owl.de>
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 9124a0d..09b2cb7 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2005-05-14 Kazu Hirata <kazu@cs.umass.edu>
+
+ PR tree-optimization/21563
+ * gcc.dg/tree-ssa/pr21563.c: New.
+
2005-05-14 Jan Hubicka <jh@suse.cz>
* gcc.dg/noreturn-7.c: Allow control reaches warning.
diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr21563.c b/gcc/testsuite/gcc.dg/tree-ssa/pr21563.c
new file mode 100644
index 0000000..b3c4588
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/tree-ssa/pr21563.c
@@ -0,0 +1,17 @@
+/* PR tree-optimization/21563
+ Make sure VRP folds the second "if" statement. */
+
+/* { dg-do compile } */
+/* { dg-options "-O2 -fno-tree-dominator-opts -fdump-tree-vrp-details" } */
+
+int
+foo (int a)
+{
+ if (a > 1)
+ if (a == 0)
+ return 1;
+ return 0;
+}
+
+/* { dg-final { scan-tree-dump-times "Folding predicate" 1 "vrp"} } */
+/* { dg-final { cleanup-tree-dump "vrp" } } */
diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c
index 8be79b5..16fa58b 100644
--- a/gcc/tree-vrp.c
+++ b/gcc/tree-vrp.c
@@ -1204,6 +1204,9 @@ compare_range_with_value (enum tree_code comp, value_range *vr, tree val)
else if (cmp == -1 || cmp == 1 || cmp == 2)
return boolean_false_node;
}
+ else if (compare_values (val, vr->min) == -1
+ || compare_values (vr->max, val) == -1)
+ return boolean_false_node;
return NULL_TREE;
}