diff mbox series

Change AArch64 specific FMAX/FMIN tests into generic MAX_EXPR/MIN_EXPR tests

Message ID 7cb9c213-c2ef-553b-a436-72feedd05fcc@arm.com
State New
Headers show
Series Change AArch64 specific FMAX/FMIN tests into generic MAX_EXPR/MIN_EXPR tests | expand

Commit Message

Szabolcs Nagy Aug. 22, 2018, 1:16 p.m. UTC
gfortran now always uses MAX_EXPR/MIN_EXPR for MAX/MIN intrinsics, so the
AArch64 specific FMAX/FMIN tests are no longer valid.

2018-08-22  Szabolcs Nagy  <szabolcs.nagy@arm.com>

	* gfortran.dg/max_fmax_aarch64.f90: Rename to...
	* gfortran.dg/max_expr.f90: ...this.
	* gfortran.dg/min_fmin_aarch64.f90: Rename to...
	* gfortran.dg/min_expr.f90: ...this.

Comments

Janne Blomqvist Aug. 22, 2018, 2:57 p.m. UTC | #1
On Wed, Aug 22, 2018 at 4:16 PM, Szabolcs Nagy <szabolcs.nagy@arm.com>
wrote:

> gfortran now always uses MAX_EXPR/MIN_EXPR for MAX/MIN intrinsics, so the
> AArch64 specific FMAX/FMIN tests are no longer valid.
>
> 2018-08-22  Szabolcs Nagy  <szabolcs.nagy@arm.com>
>
>         * gfortran.dg/max_fmax_aarch64.f90: Rename to...
>         * gfortran.dg/max_expr.f90: ...this.
>         * gfortran.dg/min_fmin_aarch64.f90: Rename to...
>         * gfortran.dg/min_expr.f90: ...this.
>

Ok, thanks!
Kyrill Tkachov Aug. 22, 2018, 3:12 p.m. UTC | #2
Hi all,

On 22/08/18 15:57, Janne Blomqvist wrote:
> On Wed, Aug 22, 2018 at 4:16 PM, Szabolcs Nagy <szabolcs.nagy@arm.com>
> wrote:
>
> > gfortran now always uses MAX_EXPR/MIN_EXPR for MAX/MIN intrinsics, so the
> > AArch64 specific FMAX/FMIN tests are no longer valid.
> >
> > 2018-08-22  Szabolcs Nagy <szabolcs.nagy@arm.com>
> >
> >         * gfortran.dg/max_fmax_aarch64.f90: Rename to...
> >         * gfortran.dg/max_expr.f90: ...this.
> >         * gfortran.dg/min_fmin_aarch64.f90: Rename to...
> >         * gfortran.dg/min_expr.f90: ...this.
> >
>
> Ok, thanks!
>

I've committed this on behalf of Szabolcs as r263778.

Thanks,
Kyrill

> -- 
> Janne Blomqvist
diff mbox series

Patch

diff --git a/gcc/testsuite/gfortran.dg/max_fmax_aarch64.f90 b/gcc/testsuite/gfortran.dg/max_expr.f90
similarity index 74%
rename from gcc/testsuite/gfortran.dg/max_fmax_aarch64.f90
rename to gcc/testsuite/gfortran.dg/max_expr.f90
index b818241a1f9..c00ad62f744 100644
--- a/gcc/testsuite/gfortran.dg/max_fmax_aarch64.f90
+++ b/gcc/testsuite/gfortran.dg/max_expr.f90
@@ -1,4 +1,4 @@ 
-! { dg-do compile { target aarch64*-*-* } }
+! { dg-do compile }
 ! { dg-options "-O2 -fdump-tree-optimized" }
 
 subroutine foo (a, b, c, d, e, f, g, h)
@@ -12,4 +12,4 @@  subroutine foof (a, b, c, d, e, f, g, h)
 end subroutine
 
 
-! { dg-final { scan-tree-dump-times "\.FMAX " 14 "optimized" } }
+! { dg-final { scan-tree-dump-times "MAX_EXPR " 14 "optimized" } }
diff --git a/gcc/testsuite/gfortran.dg/min_fmin_aarch64.f90 b/gcc/testsuite/gfortran.dg/min_expr.f90
similarity index 74%
rename from gcc/testsuite/gfortran.dg/min_fmin_aarch64.f90
rename to gcc/testsuite/gfortran.dg/min_expr.f90
index 009869b497d..5f32d5073c9 100644
--- a/gcc/testsuite/gfortran.dg/min_fmin_aarch64.f90
+++ b/gcc/testsuite/gfortran.dg/min_expr.f90
@@ -1,4 +1,4 @@ 
-! { dg-do compile { target aarch64*-*-* } }
+! { dg-do compile }
 ! { dg-options "-O2 -fdump-tree-optimized" }
 
 subroutine foo (a, b, c, d, e, f, g, h)
@@ -12,4 +12,4 @@  subroutine foof (a, b, c, d, e, f, g, h)
   a = min (a, b, c, d, e, f, g, h)
 end subroutine
 
-! { dg-final { scan-tree-dump-times "\.FMIN " 14 "optimized" } }
+! { dg-final { scan-tree-dump-times "MIN_EXPR " 14 "optimized" } }