This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[patch] Fix array overflow in gcc.dg/vect/no-vfa-vect-depend-2.c
- From: Paul Pluzhnikov <ppluzhnikov at google dot com>
- To: gcc-patches at gcc dot gnu dot org
- Cc: ppluzhnikov at google dot com
- Date: Tue, 04 Feb 2014 16:59:14 -0800
- Subject: [patch] Fix array overflow in gcc.dg/vect/no-vfa-vect-depend-2.c
- Authentication-results: sourceware.org; auth=none
Greetings,
The gcc.dg/vect/no-vfa-vect-depend-2.c failed for us, when linked with
gold, but not when linked with BFD ld.
The problem appears to be off-by-one error causing array out of bounds
access, fixed by attached patch.
OK for trunk?
Thanks,
gcc/testsuite/ChangeLog:
2014-02-04 Paul Pluzhnikov <ppluzhnikov@google.com>
* gcc.dg/vect/no-vfa-vect-depend-2.c (main1): Fix buffer
overflow.
Index: gcc/testsuite/gcc.dg/vect/no-vfa-vect-depend-2.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/no-vfa-vect-depend-2.c (revision 207487)
+++ gcc/testsuite/gcc.dg/vect/no-vfa-vect-depend-2.c (working copy)
@@ -15,7 +15,7 @@
int i;
/* Not vectorizable due to data dependence: dependence distance 1. */
- for (i = N - 1; i >= 0; i--)
+ for (i = N - 2; i >= 0; i--)
{
ia[i] = ia[i+1] * 4;
}
@@ -28,7 +28,7 @@
}
/* Vectorizable. Dependence distance -1. */
- for (i = N - 1; i >= 0; i--)
+ for (i = N - 2; i >= 0; i--)
{
ib[i+1] = ib[i] * 4;
}