Re: [PATCH, libgomp]: Require vect_simd_clones effective target for libgomp.fortran/declare-simd-[12].f90

[Uros Bizjak]

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 } }