From 5de2df7bd4a71a70ecbb3f6c514183864dc75c8a Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Sat, 14 May 2005 20:32:59 +0000 Subject: [PATCH] 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 --- gcc/ChangeLog | 5 +++++ gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/gcc.dg/tree-ssa/pr21563.c | 17 +++++++++++++++++ gcc/tree-vrp.c | 3 +++ 4 files changed, 30 insertions(+) create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/pr21563.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 23383895a30..2a1eee87285 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 Jan-Benedict Glaw diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 9124a0d8a5f..09b2cb73e0a 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2005-05-14 Kazu Hirata + + PR tree-optimization/21563 + * gcc.dg/tree-ssa/pr21563.c: New. + 2005-05-14 Jan Hubicka * 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 00000000000..b3c45886eb1 --- /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 8be79b5919c..16fa58b23b1 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; } -- 2.30.2