This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[PATCH] Fix testsuite regressions from redundant store removal patch
- From: Richard Guenther <rguenther at suse dot de>
- To: gcc-patches at gcc dot gnu dot org
- Date: Sat, 28 Mar 2009 15:13:32 +0100 (CET)
- Subject: [PATCH] Fix testsuite regressions from redundant store removal patch
This fixes
FAIL: g++.dg/warn/Warray-bounds.C (test for warnings, line 32)
FAIL: g++.dg/warn/Warray-bounds.C (test for warnings, line 43)
FAIL: g++.dg/warn/Warray-bounds.C (test for warnings, line 54)
FAIL: gcc.dg/Warray-bounds.c (test for warnings, line 32)
FAIL: gcc.dg/Warray-bounds.c (test for warnings, line 43)
FAIL: gcc.dg/Warray-bounds.c (test for warnings, line 54)
Committed to trunk.
Richard.
2009-03-28 Richard Guenther <rguenther@suse.de>
* gcc.dg/Warray-bounds.c: Do not use redundant stores.
* g++.dg/warn/Warray-bounds.C: Likewise.
Index: gcc/testsuite/gcc.dg/Warray-bounds.c
===================================================================
*** gcc/testsuite/gcc.dg/Warray-bounds.c (revision 145186)
--- gcc/testsuite/gcc.dg/Warray-bounds.c (working copy)
*************** int* f(void) {
*** 26,35 ****
a[ 9] = 0;
a[10] = 0; /* { dg-warning "array subscript" } */
a[11] = 0; /* { dg-warning "array subscript" } */
! a[2 * n() - 11] = 0; /* { dg-warning "array subscript" } */
! a[2 * n() - 10] = 0;
! a[2 * n() - 1] = 0;
! a[2 * n() - 0] = 0; /* { dg-warning "array subscript" } */
b[-1] = 0; /* { dg-warning "array subscript" } */
b[ 0] = 0;
--- 26,35 ----
a[ 9] = 0;
a[10] = 0; /* { dg-warning "array subscript" } */
a[11] = 0; /* { dg-warning "array subscript" } */
! a[2 * n() - 11] = 1; /* { dg-warning "array subscript" } */
! a[2 * n() - 10] = 1;
! a[2 * n() - 1] = 1;
! a[2 * n() - 0] = 1; /* { dg-warning "array subscript" } */
b[-1] = 0; /* { dg-warning "array subscript" } */
b[ 0] = 0;
*************** int* f(void) {
*** 37,46 ****
b[ 9] = 0;
b[10] = 0; /* { dg-warning "array subscript" } */
b[11] = 0; /* { dg-warning "array subscript" } */
! b[2 * n() - 11] = 0; /* { dg-warning "array subscript" } */
! b[2 * n() - 10] = 0;
! b[2 * n() - 1] = 0;
! b[2 * n() - 0] = 0; /* { dg-warning "array subscript" } */
c.c[-1] = 0; /* { dg-warning "array subscript" } */
c.c[ 0] = 0;
--- 37,46 ----
b[ 9] = 0;
b[10] = 0; /* { dg-warning "array subscript" } */
b[11] = 0; /* { dg-warning "array subscript" } */
! b[2 * n() - 11] = 1; /* { dg-warning "array subscript" } */
! b[2 * n() - 10] = 1;
! b[2 * n() - 1] = 1;
! b[2 * n() - 0] = 1; /* { dg-warning "array subscript" } */
c.c[-1] = 0; /* { dg-warning "array subscript" } */
c.c[ 0] = 0;
*************** int* f(void) {
*** 48,57 ****
c.c[ 9] = 0;
c.c[10] = 0; /* { dg-warning "array subscript" } */
c.c[11] = 0; /* { dg-warning "array subscript" } */
! c.c[2 * n() - 11] = 0; /* { dg-warning "array subscript" } */
! c.c[2 * n() - 10] = 0;
! c.c[2 * n() - 1] = 0;
! c.c[2 * n() - 0] = 0; /* { dg-warning "array subscript" } */
g(&a[8]);
g(&a[9]);
--- 48,57 ----
c.c[ 9] = 0;
c.c[10] = 0; /* { dg-warning "array subscript" } */
c.c[11] = 0; /* { dg-warning "array subscript" } */
! c.c[2 * n() - 11] = 1; /* { dg-warning "array subscript" } */
! c.c[2 * n() - 10] = 1;
! c.c[2 * n() - 1] = 1;
! c.c[2 * n() - 0] = 1; /* { dg-warning "array subscript" } */
g(&a[8]);
g(&a[9]);
Index: gcc/testsuite/g++.dg/warn/Warray-bounds.C
===================================================================
*** gcc/testsuite/g++.dg/warn/Warray-bounds.C (revision 145186)
--- gcc/testsuite/g++.dg/warn/Warray-bounds.C (working copy)
*************** int* f(void) {
*** 26,35 ****
a[ 9] = 0;
a[10] = 0; /* { dg-warning "array subscript" } */
a[11] = 0; /* { dg-warning "array subscript" } */
! a[2 * n() - 11] = 0; /* { dg-warning "array subscript" } */
! a[2 * n() - 10] = 0;
! a[2 * n() - 1] = 0;
! a[2 * n() - 0] = 0; /* { dg-warning "array subscript" } */
b[-1] = 0; /* { dg-warning "array subscript" } */
b[ 0] = 0;
--- 26,35 ----
a[ 9] = 0;
a[10] = 0; /* { dg-warning "array subscript" } */
a[11] = 0; /* { dg-warning "array subscript" } */
! a[2 * n() - 11] = 1; /* { dg-warning "array subscript" } */
! a[2 * n() - 10] = 1;
! a[2 * n() - 1] = 1;
! a[2 * n() - 0] = 1; /* { dg-warning "array subscript" } */
b[-1] = 0; /* { dg-warning "array subscript" } */
b[ 0] = 0;
*************** int* f(void) {
*** 37,46 ****
b[ 9] = 0;
b[10] = 0; /* { dg-warning "array subscript" } */
b[11] = 0; /* { dg-warning "array subscript" } */
! b[2 * n() - 11] = 0; /* { dg-warning "array subscript" } */
! b[2 * n() - 10] = 0;
! b[2 * n() - 1] = 0;
! b[2 * n() - 0] = 0; /* { dg-warning "array subscript" } */
c.c[-1] = 0; /* { dg-warning "array subscript" } */
c.c[ 0] = 0;
--- 37,46 ----
b[ 9] = 0;
b[10] = 0; /* { dg-warning "array subscript" } */
b[11] = 0; /* { dg-warning "array subscript" } */
! b[2 * n() - 11] = 1; /* { dg-warning "array subscript" } */
! b[2 * n() - 10] = 1;
! b[2 * n() - 1] = 1;
! b[2 * n() - 0] = 1; /* { dg-warning "array subscript" } */
c.c[-1] = 0; /* { dg-warning "array subscript" } */
c.c[ 0] = 0;
*************** int* f(void) {
*** 48,57 ****
c.c[ 9] = 0;
c.c[10] = 0; /* { dg-warning "array subscript" } */
c.c[11] = 0; /* { dg-warning "array subscript" } */
! c.c[2 * n() - 11] = 0; /* { dg-warning "array subscript" } */
! c.c[2 * n() - 10] = 0;
! c.c[2 * n() - 1] = 0;
! c.c[2 * n() - 0] = 0; /* { dg-warning "array subscript" } */
g(&a[8]);
g(&a[9]);
--- 48,57 ----
c.c[ 9] = 0;
c.c[10] = 0; /* { dg-warning "array subscript" } */
c.c[11] = 0; /* { dg-warning "array subscript" } */
! c.c[2 * n() - 11] = 1; /* { dg-warning "array subscript" } */
! c.c[2 * n() - 10] = 1;
! c.c[2 * n() - 1] = 1;
! c.c[2 * n() - 0] = 1; /* { dg-warning "array subscript" } */
g(&a[8]);
g(&a[9]);