From: Steven Bosscher Date: Thu, 14 Jul 2005 22:54:42 +0000 (+0000) Subject: re PR tree-optimization/22230 (value range propagation error) X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=3c341936db4cd869659d61387381c32fea5f01ab;p=gcc.git re PR tree-optimization/22230 (value range propagation error) PR tree-optimization/22230 gcc/ * tree-vrp.c (extract_range_from_binary_expr): Fix logics thinko in the computation of the four cross productions for "range op range". testsuite/ * gcc.dg/tree-ssa/pr22230.c: New test. From-SVN: r102038 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 1105727eb5a..2a13af21bdf 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2005-07-14 Steven Bosscher + + PR tree-optimization/22230 + * tree-vrp.c (extract_range_from_binary_expr): Fix logics thinko in + the computation of the four cross productions for "range op range". + 2005-07-14 Alexandre Oliva Ulrich Weigand diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 334e96e4638..2e5eee9ccf4 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2005-07-15 Steven Bosscher + + PR tree-optimization/22230 + * gcc.dg/tree-ssa/pr22230.c: New test. + 2005-07-14 Jakub Jelinek * gfortran.dg/g77/cpp6.f: New test. diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr22230.c b/gcc/testsuite/gcc.dg/tree-ssa/pr22230.c new file mode 100644 index 00000000000..4d653490934 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr22230.c @@ -0,0 +1,21 @@ +/* { dg-do run } */ +/* { dg-options "-O1 -ftree-vrp" } */ + +/* PR tree-optimization/22230 + + The meet of the ranges in "i*i" was not computed correctly, leading + gcc to believe that a was equal to 0 after the loop. */ + +extern void abort (void) __attribute__((noreturn)); + +int main (void) +{ + long a, i; + + for (i = 0; i < 5; i++) + a = i * i; + if (a != 16) + abort (); + return 0; +} + diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index bcbc7384d2e..a42a21a882e 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -1183,7 +1183,7 @@ extract_range_from_binary_expr (value_range_t *vr, tree expr) ? vrp_int_const_binop (code, vr0.max, vr1.min) : NULL_TREE; - val[3] = (vr0.min != vr1.min && vr0.max != vr1.max) + val[3] = (vr0.min != vr0.max && vr1.min != vr1.max) ? vrp_int_const_binop (code, vr0.max, vr1.max) : NULL_TREE;