From 08303d5bb44c250c178aeb53b70027a9ad188bb8 Mon Sep 17 00:00:00 2001 From: "James A. Morrison" Date: Tue, 2 Aug 2005 05:08:40 +0000 Subject: [PATCH] re PR tree-optimization/23129 (VRP propagates division of antiranges incorrectly) 2005-08-02 James A. Morrison PR tree-optimization/23129 * tree-vrp.c (extract_range_from_binary_expr): Set value range to varying for divisions with anti-ranges. From-SVN: r102653 --- gcc/ChangeLog | 6 ++++++ gcc/testsuite/ChangeLog | 4 ++++ gcc/testsuite/gcc.c-torture/execute/vrp-4.c | 20 ++++++++++++++++++++ gcc/tree-vrp.c | 3 ++- 4 files changed, 32 insertions(+), 1 deletion(-) create mode 100644 gcc/testsuite/gcc.c-torture/execute/vrp-4.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 3be4e21..430e474 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2005-08-02 James A. Morrison + + PR tree-optimization/23129 + * tree-vrp.c (extract_range_from_binary_expr): Set value range to + varying for divisions with anti-ranges. + 2005-08-02 Jan Hubicka * tree-ssa-dom.c (thread_across_edge): Remove updating here. diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 2efa499..0060ae6 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2005-08-02 James A. Morrison + + * gcc.c-torture/execute/vrp-4.c: New test. + 2005-08-02 Jan Hubicka * update-threading.c: New test. diff --git a/gcc/testsuite/gcc.c-torture/execute/vrp-4.c b/gcc/testsuite/gcc.c-torture/execute/vrp-4.c new file mode 100644 index 0000000..23f2922 --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/execute/vrp-4.c @@ -0,0 +1,20 @@ +extern void exit (int); +extern void abort (); + +void test(int x, int y) +{ + int c; + + if (x == 1) abort(); + if (y == 1) abort(); + + c = x / y; + + if (c != 1) abort(); +} + +int main() +{ + test(2, 2); + exit (0); +} diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index 68960e5..68bcb1e 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -1136,7 +1136,8 @@ extract_range_from_binary_expr (value_range_t *vr, tree expr) the new range. */ /* Divisions by zero result in a VARYING value. */ - if (code != MULT_EXPR && range_includes_zero_p (&vr1)) + if (code != MULT_EXPR + && (vr0.type == VR_ANTI_RANGE || range_includes_zero_p (&vr1))) { set_value_range_to_varying (vr); return; -- 2.7.4