Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-05 Thread Mikael Morin
On Thursday 04 August 2011 23:46:08 Steve Kargl wrote: > On Thu, Aug 04, 2011 at 11:36:11PM +0200, Mikael Morin wrote: > > On Thursday 04 August 2011 22:45:34 Steve Kargl wrote: > > > On Tue, Aug 02, 2011 at 05:34:46PM +0200, Tobias Burnus wrote: > > > > There might be still some missing or wrong c

Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-04 Thread Steve Kargl
On Thu, Aug 04, 2011 at 11:36:11PM +0200, Mikael Morin wrote: > On Thursday 04 August 2011 22:45:34 Steve Kargl wrote: > > On Tue, Aug 02, 2011 at 05:34:46PM +0200, Tobias Burnus wrote: > > > There might be still some missing or wrong cleanups, but most remaining > > > .mod files are from gfortran.

Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-04 Thread Mikael Morin
On Thursday 04 August 2011 22:45:34 Steve Kargl wrote: > On Tue, Aug 02, 2011 at 05:34:46PM +0200, Tobias Burnus wrote: > > There might be still some missing or wrong cleanups, but most remaining > > .mod files are from gfortran.fortran-torture. > > Yes, you missed a few. And you missed my two pa

Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-04 Thread Mikael Morin
On Wednesday 03 August 2011 22:55:34 Tobias Burnus wrote: > Mikael Morin wrote: > > Patch updated. > > Looks good! Thanks for the further clean up! > Committed as revision 177415. I also added the gfortran.dg/ directory in the ChangeLog for you previous patch (as revision 177416). Mikael Index

Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-04 Thread Steve Kargl
On Tue, Aug 02, 2011 at 05:34:46PM +0200, Tobias Burnus wrote: > > There might be still some missing or wrong cleanups, but most remaining > .mod files are from gfortran.fortran-torture. > Yes, you missed a few. 2011-08-04 Steven G. Kargl * gfortran.dg/trim_optimize_2.f90: Clean up

Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-03 Thread Tobias Burnus
Mikael Morin wrote: Patch updated. Looks good! Thanks for the further clean up! Tobias

Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-03 Thread Mikael Morin
On Wednesday 03 August 2011 18:22:55 Tobias Burnus wrote: > > --- pr32921.f (révision 177274) > > +++ pr32921.f (copie de travail) > > @@ -46,5 +46,5 @@ > > > > RETURN > > END > > > > ! { dg-final { scan-tree-dump-times "stride" 4 "lim1" } } > > > > -! { dg-final

Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-03 Thread Tobias Burnus
On 08/03/2011 05:55 PM, Mikael Morin wrote: Tobias, I would like your input on lto/pr47839_{0,1}.f90 for which you put the cleanup directive for module pec_mod from pr47839_1.f90 in the pr47839_0.f90 file. As it seems to be an overlook, this patches changes it. Is that correct? I think either

Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-03 Thread Mikael Morin
On Tuesday 02 August 2011 18:01:35 Mikael Morin wrote: > Thanks. I'll see if I have additional fixes after updating. Here they are. Tobias, I would like your input on lto/pr47839_{0,1}.f90 for which you put the cleanup directive for module pec_mod from pr47839_1.f90 in the pr47839_0.f90 file. As

Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-02 Thread Tobias Burnus
On 08/02/2011 06:01 PM, Mikael Morin wrote: It's a pity that I've made the same fixes myself. I was planing to commit soon. :-( Indeed it would have been more useful to avoid replicating the work. :-( On Tuesday 02 August 2011 17:34:46 Tobias Burnus wrote: * Wrong case: All module files are

Re: [Patch, Fortran testsuite, committed] Add/fix dg-final cleanup-module

2011-08-02 Thread Mikael Morin
It's a pity that I've made the same fixes myself. I was planing to commit soon. :-( On Tuesday 02 August 2011 17:34:46 Tobias Burnus wrote: > This patch fixes dg-final cleanup-modules issues: > * Missing cleanups > * Wrong case: All module files are lower case This is not actually a problem as cl