Re: [PATCH] aarch64: fortran: Adjust vect-8.f90 for libmvec

[Richard Sandiford]

Szabolcs Nagy <szabolcs.n...@arm.com> writes:
> With new glibc one more loop can be vectorized via simd exp in libmvec.
>
> Found by the Linaro TCWG CI.
>
> gcc/testsuite/ChangeLog:
>
>       * gfortran/vect/vect-8.f90: Accept more vectorized loops.

OK.  At first I thought it would be good to "defend" the increase when
it's supposed to apply, but it would need a relatively complicated check,
and there should be plenty of test coverage elsewhere.

Thanks,
Richard

> ---
>  gcc/testsuite/gfortran.dg/vect/vect-8.f90 | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gcc/testsuite/gfortran.dg/vect/vect-8.f90 
> b/gcc/testsuite/gfortran.dg/vect/vect-8.f90
> index ca72ddcffca..938dfc29754 100644
> --- a/gcc/testsuite/gfortran.dg/vect/vect-8.f90
> +++ b/gcc/testsuite/gfortran.dg/vect/vect-8.f90
> @@ -704,7 +704,7 @@ CALL track('KERNEL  ')
>  RETURN
>  END SUBROUTINE kernel
>  
> -! { dg-final { scan-tree-dump-times "vectorized 25 loops" 1 "vect" { target 
> aarch64_sve } } }
> -! { dg-final { scan-tree-dump-times "vectorized 24 loops" 1 "vect" { target 
> { aarch64*-*-* && { ! aarch64_sve } } } } }
> +! { dg-final { scan-tree-dump-times "vectorized 2\[56\] loops" 1 "vect" { 
> target aarch64_sve } } }
> +! { dg-final { scan-tree-dump-times "vectorized 2\[45\] loops" 1 "vect" { 
> target { aarch64*-*-* && { ! aarch64_sve } } } } }
>  ! { dg-final { scan-tree-dump-times "vectorized 2\[234\] loops" 1 "vect" { 
> target { vect_intdouble_cvt && { ! aarch64*-*-* } } } } }
>  ! { dg-final { scan-tree-dump-times "vectorized 17 loops" 1 "vect" { target 
> { { ! vect_intdouble_cvt } && { ! aarch64*-*-* } } } } }