This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] gfortran.dg/vect/vect-5: don't xfail lp64


howarth@bromo.msbb.uc.edu (Jack Howarth) wrote on 19/09/2006 04:57:00 AM:

> Since we are now passing the gfortran.dg/vect/vect-5.f90
> testcase on 64-bit platforms, stop xfailing lp64.
>
> http://gcc.gnu.org/ml/gcc/2006-09/msg00330.html
>
> Tested on Darwin PPC at -m64 and on x86_64.
> Okay for gcc trunk?
>                     Jack
>

Looks ok to me, except - did you intentionally leave out this line?:
+! { dg-final { scan-tree-dump-times "vectorized 2 loops" 1 "vect" } }

>From what you said before, sounds like the correct solution would be to
also add this line, and figure out why ia64 fails this check (or open a PR
for that (for ia64) in the meantime).

thanks!

dorit

>
> 2006-09-14   Jack Howarth  <howarth@bromo.med.uc.edu>
>
>    * gfortran.dg/vect/vect-5.f90: Don't xfail lp64.
>
> Index: gcc/testsuite/gfortran.dg/vect/vect-5.f90
> ===================================================================
> --- gcc/testsuite/gfortran.dg/vect/vect-5.f90   (revision 117048)
> +++ gcc/testsuite/gfortran.dg/vect/vect-5.f90   (working copy)
> @@ -35,9 +35,8 @@
>          stop
>          end
>
> -! { dg-final { scan-tree-dump-times "vectorized 2 loops" 1 "vect" {
> xfail { lp64 } } } }
> -! { dg-final { scan-tree-dump-times "Alignment of access forced
> using peeling" 1 "vect" { xfail { vect_no_align || lp64 } } } }
> -! { dg-final { scan-tree-dump-times "Vectorizing an unaligned
> access" 2 "vect" { xfail { vect_no_align || lp64 } } } }
> +! { dg-final { scan-tree-dump-times "Alignment of access forced
> using peeling" 1 "vect" { xfail { vect_no_align } } } }
> +! { dg-final { scan-tree-dump-times "Vectorizing an unaligned
> access" 2 "vect" { xfail { vect_no_align } } } }
>  ! { dg-final { scan-tree-dump-times "Alignment of access forced
> using versioning." 3 "vect" { target { ilp32 && vect_no_align } } } }
>
>  ! We also expect to vectorize one loop for lp64 targets that support


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]