Re: [PATCH 2/2] Optimize fortran loops with +-1 step.

2016-07-12 Thread Martin Liška
On 07/12/2016 12:07 PM, Richard Biener wrote: > That change is certainly spurious (not in ChangeLog) and bogus given the > comment before the scan. > > Richard. Hi. I'll revert the change in ldist-1.f90 which I accidentally committed a week ago, it's going to be part of second version of the pat

Re: [PATCH 2/2] Optimize fortran loops with +-1 step.

2016-07-12 Thread Richard Biener
On Sun, Jul 10, 2016 at 9:37 AM, Andreas Schwab wrote: > marxin writes: > >> diff --git a/gcc/testsuite/gfortran.dg/ldist-1.f90 >> b/gcc/testsuite/gfortran.dg/ldist-1.f90 >> index ea3990d..2030328 100644 >> --- a/gcc/testsuite/gfortran.dg/ldist-1.f90 >> +++ b/gcc/testsuite/gfortran.dg/ldist-1.f9

Re: [PATCH 2/2] Optimize fortran loops with +-1 step.

2016-07-10 Thread Andreas Schwab
marxin writes: > diff --git a/gcc/testsuite/gfortran.dg/ldist-1.f90 > b/gcc/testsuite/gfortran.dg/ldist-1.f90 > index ea3990d..2030328 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 lega

Re: [PATCH 2/2] Optimize fortran loops with +-1 step.

2016-07-08 Thread Martin Liška
On 07/07/2016 05:53 PM, Tobias Burnus wrote: > On Thu, Jul 07, 2016 at 02:13:12PM +0200, Tobias Burnus wrote: >> marxin wrote: >>> gcc/fortran/ChangeLog: >>> >>> 2016-07-01 Martin Liska >>> * lang.opt (Wundefined-do-loop): New option. >>>* resolve.c (gfc_resolve_iterator): Warn for W

Re: [PATCH 2/2] Optimize fortran loops with +-1 step.

2016-07-07 Thread Tobias Burnus
On Thu, Jul 07, 2016 at 02:13:12PM +0200, Tobias Burnus wrote: > marxin wrote: > > gcc/fortran/ChangeLog: > > > > 2016-07-01 Martin Liska > > * lang.opt (Wundefined-do-loop): New option. > >* resolve.c (gfc_resolve_iterator): Warn for Wundefined-do-loop. > > (gfc_trans_simple_do

Re: [PATCH 2/2] Optimize fortran loops with +-1 step.

2016-07-07 Thread Tobias Burnus
marxin wrote: > gcc/fortran/ChangeLog: > > 2016-07-01 Martin Liska > * lang.opt (Wundefined-do-loop): New option. >* resolve.c (gfc_resolve_iterator): Warn for Wundefined-do-loop. > (gfc_trans_simple_do): Generate a c-style loop. > (gfc_trans_do): Fix GNU coding style.

[PATCH 2/2] Optimize fortran loops with +-1 step.

2016-07-07 Thread marxin
gcc/testsuite/ChangeLog: 2016-07-01 Martin Liska * gfortran.dg/do_1.f90: Remove a corner case that triggers an undefined behavior. * gfortran.dg/do_3.F90: Likewise. * gfortran.dg/do_check_11.f90: New test. * gfortran.dg/do_check_12.f90: New test.