From 90c6f26c8bc51f00827fcaa4b768d97ebf720ef5 Mon Sep 17 00:00:00 2001 From: Richard Biener Date: Tue, 2 Feb 2016 15:19:32 +0000 Subject: [PATCH] re PR tree-optimization/69595 (Bogus -Warray-bound warning due to missed optimization) 2016-02-02 Richard Biener PR tree-optimization/69595 * match.pd: Add range test simplifications to true/false. * gcc.dg/Warray-bounds-17.c: New testcase. From-SVN: r233076 --- gcc/ChangeLog | 5 +++++ gcc/match.pd | 18 ++++++++++++++++++ gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/gcc.dg/Warray-bounds-17.c | 13 +++++++++++++ 4 files changed, 41 insertions(+) create mode 100644 gcc/testsuite/gcc.dg/Warray-bounds-17.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 9a2cec89cbe..3b548f56a9d 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2016-02-02 Richard Biener + + PR tree-optimization/69595 + * match.pd: Add range test simplifications to true/false. + 2016-02-02 Thomas Schwinge * omp-builtins.def (BUILT_IN_GOACC_HOST_DATA): Remove. diff --git a/gcc/match.pd b/gcc/match.pd index 491f769c1dd..6c8ebd5e090 100644 --- a/gcc/match.pd +++ b/gcc/match.pd @@ -2094,6 +2094,24 @@ DEFINE_INT_AND_FLOAT_ROUND_FN (RINT) (bit_and:c (ordered @0 @0) (ordered:c@2 @0 @1)) @2) +/* Simple range test simplifications. */ +/* A < B || A >= B -> true. */ +(for test1 (lt le ne) + test2 (ge gt eq) + (simplify + (bit_ior:c (test1 @0 @1) (test2 @0 @1)) + (if (INTEGRAL_TYPE_P (TREE_TYPE (@0)) + || VECTOR_INTEGER_TYPE_P (TREE_TYPE (@0))) + { constant_boolean_node (true, type); }))) +/* A < B && A >= B -> false. */ +(for test1 (lt lt lt le ne eq) + test2 (ge gt eq gt eq gt) + (simplify + (bit_and:c (test1 @0 @1) (test2 @0 @1)) + (if (INTEGRAL_TYPE_P (TREE_TYPE (@0)) + || VECTOR_INTEGER_TYPE_P (TREE_TYPE (@0))) + { constant_boolean_node (false, type); }))) + /* -A CMP -B -> B CMP A. */ (for cmp (tcc_comparison) scmp (swapped_tcc_comparison) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 9d89eee0ebb..9ed6c54535b 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2016-02-02 Richard Biener + + PR tree-optimization/69595 + * gcc.dg/Warray-bounds-17.c: New testcase. + 2016-02-02 Richard Biener PR tree-optimization/69606 diff --git a/gcc/testsuite/gcc.dg/Warray-bounds-17.c b/gcc/testsuite/gcc.dg/Warray-bounds-17.c new file mode 100644 index 00000000000..e790037feb1 --- /dev/null +++ b/gcc/testsuite/gcc.dg/Warray-bounds-17.c @@ -0,0 +1,13 @@ +/* { dg-do compile } */ +/* { dg-options "-O2 -Warray-bounds" } */ + +char *y; +void foo (int sysnum) +{ + static char *x[] = {}; + int nsyscalls = sizeof x / sizeof x[0]; + if (sysnum < 0 || sysnum >= nsyscalls) + return; + else + y = x[sysnum]; /* { dg-bogus "above array bounds" } */ +} -- 2.30.2