[Bug testsuite/88265] [9 Regression] gfortran.dg/simd-builtins-1.f90 fails when using --with-arch=...

2018-11-30 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88265

--- Comment #7 from Martin Liška  ---
Author: marxin
Date: Fri Nov 30 10:48:51 2018
New Revision: 20

URL: https://gcc.gnu.org/viewcvs?rev=20=gcc=rev
Log:
Restrict vector extension to use 128bit vectors (PR testsuite/88265).

2018-11-30  Martin Liska  

PR testsuite/88265
* gfortran.dg/simd-builtins-1.f90: Restrict to 128b vectors.
* gfortran.dg/simd-builtins-6.f90: Likewise.

Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gfortran.dg/simd-builtins-1.f90
trunk/gcc/testsuite/gfortran.dg/simd-builtins-6.f90

[Bug testsuite/88265] [9 Regression] gfortran.dg/simd-builtins-1.f90 fails when using --with-arch=...

2018-11-30 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88265

Martin Liška  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #8 from Martin Liška  ---
Fixed.

[Bug testsuite/88265] [9 Regression] gfortran.dg/simd-builtins-1.f90 fails when using --with-arch=...

2018-11-30 Thread janus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88265

--- Comment #6 from janus at gcc dot gnu.org ---
(In reply to Jakub Jelinek from comment #5)
> Instead of adding -march= please add instead say -mavx2 -mno-avx512f or
> whatever ISA set you want to pick, and probably also
> -mprefer-vector-width=256 or what you want.

I guess it might be a good idea to have different variants of the test case
that check the correct vector widths for AVX, AVX2, AVX512, etc. Of course one
would need to make sure that e.g. the AVX512 variant is only executed on AVX512
hardware (e.g. skylake CPU).