Message ID | 86a3b430-0282-305b-8796-5a696d53b46a@suse.cz |
---|---|
State | New |
Headers | show |
On 07/08/2016 08:26 AM, Martin Liška wrote: > Hello > > Following patch fixes fallout caused by the patch set: > https://gcc.gnu.org/ml/gcc-regression/2016-07/msg00097.html > > Ready after it finished regression tests? > Thanks, > Martin > > > 0001-Fix-Fortran-DO-loop-fallback.patch > > > From c5dd7ad62f795cce560c7f1bb8767b7ed9298d8a Mon Sep 17 00:00:00 2001 > From: marxin <mliska@suse.cz> > Date: Fri, 8 Jul 2016 15:51:54 +0200 > Subject: [PATCH] Fix Fortran DO loop fallback > > gcc/testsuite/ChangeLog: > > 2016-07-08 Martin Liska <mliska@suse.cz> > > * gfortran.dg/ldist-1.f90: Update expected dump scan. > * gfortran.dg/pr42108.f90: Likewise. > * gfortran.dg/vect/pr62283.f: Likewise. Shouldn't ldist-1.f90 be scan-tree-dump-not? It seems like you change it from that just last week, but it wasn't mentioned in the ChangeLog. OK with that change. jeff
> On Jul 11, 2016, at 7:44 AM, Jeff Law <law@redhat.com> wrote: > > On 07/08/2016 08:26 AM, Martin Liška wrote: >> Hello >> >> Following patch fixes fallout caused by the patch set: >> https://gcc.gnu.org/ml/gcc-regression/2016-07/msg00097.html >> >> Ready after it finished regression tests? >> Thanks, >> Martin >> >> >> 0001-Fix-Fortran-DO-loop-fallback.patch >> >> >> From c5dd7ad62f795cce560c7f1bb8767b7ed9298d8a Mon Sep 17 00:00:00 2001 >> From: marxin <mliska@suse.cz> >> Date: Fri, 8 Jul 2016 15:51:54 +0200 >> Subject: [PATCH] Fix Fortran DO loop fallback >> >> gcc/testsuite/ChangeLog: >> >> 2016-07-08 Martin Liska <mliska@suse.cz> >> >> * gfortran.dg/ldist-1.f90: Update expected dump scan. >> * gfortran.dg/pr42108.f90: Likewise. >> * gfortran.dg/vect/pr62283.f: Likewise. > Shouldn't ldist-1.f90 be scan-tree-dump-not? It seems like you change it from that just last week, but it wasn't mentioned in the ChangeLog. > > OK with that change. A gentle reminder, fortran patches should include the fortran list...
From c5dd7ad62f795cce560c7f1bb8767b7ed9298d8a Mon Sep 17 00:00:00 2001 From: marxin <mliska@suse.cz> Date: Fri, 8 Jul 2016 15:51:54 +0200 Subject: [PATCH] Fix Fortran DO loop fallback gcc/testsuite/ChangeLog: 2016-07-08 Martin Liska <mliska@suse.cz> * gfortran.dg/ldist-1.f90: Update expected dump scan. * gfortran.dg/pr42108.f90: Likewise. * gfortran.dg/vect/pr62283.f: Likewise. --- gcc/testsuite/gfortran.dg/ldist-1.f90 | 2 +- gcc/testsuite/gfortran.dg/pr42108.f90 | 2 -- gcc/testsuite/gfortran.dg/vect/pr62283.f | 2 +- 3 files changed, 2 insertions(+), 4 deletions(-) diff --git a/gcc/testsuite/gfortran.dg/ldist-1.f90 b/gcc/testsuite/gfortran.dg/ldist-1.f90 index 2030328..071a651 100644 --- a/gcc/testsuite/gfortran.dg/ldist-1.f90 +++ b/gcc/testsuite/gfortran.dg/ldist-1.f90 @@ -32,4 +32,4 @@ end Subroutine PADEC ! There are 5 legal partitions in this code. Based on the data ! locality heuristic, this loop should not be split. -! { dg-final { scan-tree-dump "distributed: split to" "ldist" } } +! { dg-final { scan-tree-dump-times "distributed: split to" 0 "ldist" } } diff --git a/gcc/testsuite/gfortran.dg/pr42108.f90 b/gcc/testsuite/gfortran.dg/pr42108.f90 index eb93604..a913aa4 100644 --- a/gcc/testsuite/gfortran.dg/pr42108.f90 +++ b/gcc/testsuite/gfortran.dg/pr42108.f90 @@ -21,7 +21,5 @@ subroutine eval(foo1,foo2,foo3,foo4,x,n,nnd) end do end subroutine eval -! We should have hoisted the division -! { dg-final { scan-tree-dump "in all uses of countm1\[^\n\]* / " "pre" } } ! There should be only one load from n left ! { dg-final { scan-tree-dump-times "\\*n_" 1 "fre1" } } diff --git a/gcc/testsuite/gfortran.dg/vect/pr62283.f b/gcc/testsuite/gfortran.dg/vect/pr62283.f index 7df3d99..2933f51 100644 --- a/gcc/testsuite/gfortran.dg/vect/pr62283.f +++ b/gcc/testsuite/gfortran.dg/vect/pr62283.f @@ -13,4 +13,4 @@ C { dg-additional-options "-fvect-cost-model=dynamic" } beta=3.141593 y=y+beta*x end -C { dg-final { scan-tree-dump-times "vectorized 1 loops" 2 "vect" { target { vect_hw_misalign } } } } +C { dg-final { scan-tree-dump "vectorized 1 loops" "vect" { target { vect_hw_misalign } } } } -- 2.8.4