From patchwork Wed Aug 18 14:05:04 2010
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Fix vectorization test case.
Date: Wed, 18 Aug 2010 04:05:04 -0000
From: tejas belagod <tejas.belagod@arm.com>
X-Patchwork-Id: 62042
Message-Id: <1282140304.15098.5.camel@e102484-lin.cambridge.arm.com>
To: gcc-patches@gcc.gnu.org

Hi,

One of the loops in gcc/testsuite/gcc.dg/vect/vect-1.c is suspect. A
two-level nested loop in foo() uses the same index variable for both
loops, but uses a different variable to access the matrix inside the
loop. I presume this is a typo as this loop is expected to vectorize.
Attached is a patch that fixes this.

---
Tejas Belagod
ARM.

gcc/testsuite

2010-08-18 Tejas Belagod <tejas.belagod@arm.com>

	* gcc.dg/vect/vect-1.c (foo): Change loop index variable.

Index: vect-1.c
===================================================================
--- vect-1.c	(revision 163337)
+++ vect-1.c	(working copy)
@@ -37,7 +37,7 @@
   /* Vectorizable.  */
   diff = 0;
   for (i = 0; i < N; i++) {
-    for (i = 0; i < N; i++) {
+    for (j = 0; j < N; j++) {
       diff += (image[i][j] - block[i][j]);
     }
   }