Hi,

When I backported support for unaligned vector load/store operations on
POWER8 to GCC 4.8, I fumbled the change for gcc.dg/vect/vect-33.c.  One
of the original tests was:

/* { dg-final { scan-tree-dump-times "Vectorizing an unaligned access" 0 "vect" 
} } */ 

which I modified to

/* { dg-final { scan-tree-dump-times "Vectorizing an unaligned access" 0 "vect" 
{ target { ! vect_hw_misalign } } } } */

This caused the test to be skipped for architectures other than PowerPC,
which was a mistake.  The correct test should have been:

/* { dg-final { scan-tree-dump-times "Vectorizing an unaligned access" 0 "vect" 
{ target { { ! powerpc*-*-* } || { ! vect_hw_misalign } } } } } */

which leaves things alone for other architectures.

Ok for 4.8?

Thanks,
Bill


2015-06-15  Bill Schmidt  <wschm...@linux.vnet.ibm.com>

        * gcc.dg/vect/vect-33.c: Don't exclude "Vectorizing an unaligned
        access" test for non-PowerPC arches.


Index: gcc/testsuite/gcc.dg/vect/vect-33.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/vect-33.c (revision 224490)
+++ gcc/testsuite/gcc.dg/vect/vect-33.c (working copy)
@@ -38,7 +38,7 @@ int main (void)
 
 
 /* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect"  } } */
-/* { dg-final { scan-tree-dump-times "Vectorizing an unaligned access" 0 
"vect" { target { ! vect_hw_misalign } } } } */
+/* { dg-final { scan-tree-dump-times "Vectorizing an unaligned access" 0 
"vect" { target { { ! powerpc*-*-* } || { ! vect_hw_misalign } } } } } */
 /* { dg-final { scan-tree-dump "Alignment of access forced using peeling" 
"vect" { target vector_alignment_reachable } } } */
 /* { dg-final { scan-tree-dump-times "Alignment of access forced using 
versioning" 1 "vect" { target { {! vector_alignment_reachable} && {! 
vect_hw_misalign} } } } } */
 /* { dg-final { cleanup-tree-dump "vect" } } */


Reply via email to