diff options
| -rw-r--r-- | gcc/ChangeLog | 4 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 4 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/Warray-bounds-11.c | 12 | ||||
| -rw-r--r-- | gcc/tree-vrp.c | 5 |
4 files changed, 23 insertions, 2 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 787040f..cbc6f8b 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,7 @@ +2013-08-30 Meador Inge <meadori@codesourcery.com> + + * tree-vrp.c (check_array_ref): Bail out on zero-length arrays. + 2013-08-30 Marek Polacek <polacek@redhat.com> * Makefile.in (ubsan.o): Add. diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 4895bb8..144a6e6 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2013-08-30 Meador Inge <meadori@codesourcery.com> + + * gcc.dg/Warray-bounds-11.c: New testcase. + 2013-08-30 Marek Polacek <polacek@redhat.com> * g++.dg/ubsan/div-by-zero-1.C: New test. diff --git a/gcc/testsuite/gcc.dg/Warray-bounds-11.c b/gcc/testsuite/gcc.dg/Warray-bounds-11.c new file mode 100644 index 0000000..130175d --- /dev/null +++ b/gcc/testsuite/gcc.dg/Warray-bounds-11.c @@ -0,0 +1,12 @@ +/* { dg-do compile } */ +/* { dg-options "-O2 -Warray-bounds -std=gnu99" } */ +/* Test zero-length arrays for GNU C. */ + +unsigned int a[] = { }; +unsigned int size_a; + +int test(void) +{ + /* This should not warn. */ + return size_a ? a[0] : 0; +} diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index d5548ff..062e03c 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -6137,9 +6137,10 @@ check_array_ref (location_t location, tree ref, bool ignore_off_by_one) low_sub = up_sub = TREE_OPERAND (ref, 1); up_bound = array_ref_up_bound (ref); - /* Can not check flexible arrays. */ + /* Can not check flexible arrays or zero-length arrays. */ if (!up_bound - || TREE_CODE (up_bound) != INTEGER_CST) + || TREE_CODE (up_bound) != INTEGER_CST + || tree_int_cst_equal (up_bound, integer_minus_one_node)) return; /* Accesses to trailing arrays via pointers may access storage |