diff options
author | Meador Inge <meadori@codesourcery.com> | 2013-08-30 16:36:46 +0000 |
---|---|---|
committer | Meador Inge <meadori@gcc.gnu.org> | 2013-08-30 16:36:46 +0000 |
commit | 0a6ea5c9c9fb6fdc077bedeba45446c809bb6850 (patch) | |
tree | e4bdf9b8eefe7d0700bc6cb490ad32d9a3766dae /gcc | |
parent | de5a5fa1395db2cb5da4d0593fef40ec22378576 (diff) | |
download | gcc-0a6ea5c9c9fb6fdc077bedeba45446c809bb6850.zip gcc-0a6ea5c9c9fb6fdc077bedeba45446c809bb6850.tar.gz gcc-0a6ea5c9c9fb6fdc077bedeba45446c809bb6850.tar.bz2 |
tree-vrp.c (check_array_ref): Bail out on zero-length arrays.
gcc/
2013-08-30 Meador Inge <meadori@codesourcery.com>
* tree-vrp.c (check_array_ref): Bail out on zero-length arrays.
gcc/testsuite/
2013-08-30 Meador Inge <meadori@codesourcery.com>
* gcc.dg/Warray-bounds-11.c: New testcase.
From-SVN: r202115
Diffstat (limited to 'gcc')
-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 |