diff options
author | Kazu Hirata <kazu@cs.umass.edu> | 2005-05-14 20:32:59 +0000 |
---|---|---|
committer | Kazu Hirata <kazu@gcc.gnu.org> | 2005-05-14 20:32:59 +0000 |
commit | 5de2df7bd4a71a70ecbb3f6c514183864dc75c8a (patch) | |
tree | d87d80e20a98cc3925d2abf5c81ae16134950b55 | |
parent | 92df92cddae0295ede176f971612090ef32dfbf9 (diff) | |
download | gcc-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/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/tree-ssa/pr21563.c | 17 | ||||
-rw-r--r-- | gcc/tree-vrp.c | 3 |
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; } |