On Tue, May 27, 2014 at 9:18 AM, Jakub Jelinek <ja...@redhat.com> wrote:
> Please don't remove the dg-additional-options there, that is completely
> intentional there, only the simd clones are built for SSE2/AVX/AVX2,
> the simd loops are built with whatever options the loop is compiled with,
> and for the common case (AVX or later HW, but compiler not configured to
> support only AVX or later) I want to test as much vectorization as possible.
> Requiring vect_simd_clone or the whitespace change is fine, though I'd
> just use ! { dg-do run { target vect_simd_clones } } instead of
> dg-require-effective-target.
Thanks for the explanation.
Following is the v2 patch that I plan to commit after testing:
2014-05-27 Uros Bizjak <ubiz...@gmail.com>
* testsuite/libgomp.fortran/declare-simd-1.f90: Require
vect_simd_clones effective target.
* testsuite/libgomp.fortran/declare-simd-2.f90: Ditto.
Uros.
Index: testsuite/libgomp.fortran/declare-simd-1.f90
===================================================================
--- testsuite/libgomp.fortran/declare-simd-1.f90 (revision 210956)
+++ testsuite/libgomp.fortran/declare-simd-1.f90 (working copy)
@@ -1,3 +1,4 @@
+! { dg-do run { target vect_simd_clones } }
! { dg-options "-fno-inline" }
! { dg-additional-options "-msse2" { target sse2_runtime } }
! { dg-additional-options "-mavx" { target avx_runtime } }
Index: testsuite/libgomp.fortran/declare-simd-2.f90
===================================================================
--- testsuite/libgomp.fortran/declare-simd-2.f90 (revision 210956)
+++ testsuite/libgomp.fortran/declare-simd-2.f90 (working copy)
@@ -1,6 +1,6 @@
-! { dg-do run }
+! { dg-do run { target vect_simd_clones } }
! { dg-options "-fno-inline" }
- ! { dg-additional-sources declare-simd-3.f90 }
+! { dg-additional-sources declare-simd-3.f90 }
! { dg-additional-options "-msse2" { target sse2_runtime } }
! { dg-additional-options "-mavx" { target avx_runtime } }
