Re: [Patch, fortran] PR 25708 Reduce seeks when loading module files

2011-12-01 Thread Mikael Morin
On Wednesday 30 November 2011 23:49:58 Janne Blomqvist wrote: > > With the updated patch, the number of lseek's when compiling > > aermod.f90 drop to 38, which is a factor of 15 reduction compared > > to the current trunk. And a factor of 55 compared to trunk a few days > > ago before Thomas' p

Re: [Patch, fortran] PR 25708 Reduce seeks when loading module files

2011-11-30 Thread Janne Blomqvist
Dominique pointed out an extra unused variable in peek_atom(). Updated patch attached On Thu, Dec 1, 2011 at 00:04, Janne Blomqvist wrote: > On Wed, Nov 30, 2011 at 22:36, Mikael Morin wrote: >> Hi, >> >>> >>> modseek1.diff >>>   diff --git a/gcc/fortran/module.c b/gcc/fortran/module.c >>> index

Re: [Patch, fortran] PR 25708 Reduce seeks when loading module files

2011-11-30 Thread Janne Blomqvist
On Wed, Nov 30, 2011 at 22:36, Mikael Morin wrote: > Hi, > >> >> modseek1.diff >>   diff --git a/gcc/fortran/module.c b/gcc/fortran/module.c >> index 70f8565..982425d 100644 >> --- a/gcc/fortran/module.c >> +++ b/gcc/fortran/module.c >> @@ -1069,51 +1069,49 @@ module_unget_char (void) >>  static v

Re: [Patch, fortran] PR 25708 Reduce seeks when loading module files

2011-11-30 Thread Mikael Morin
On Wednesday 30 November 2011 18:32:11 Janne Blomqvist wrote: > Hi, > > this patch expands a bit on the recent work done by Thomas Koenig. > Using aermod.f90 from the polyhedron benchmark suite as a test case, > the lseek() calls as reported by strace -c -f go roughly as > > - trunk before Thomas

[Patch, fortran] PR 25708 Reduce seeks when loading module files

2011-11-30 Thread Janne Blomqvist
Hi, this patch expands a bit on the recent work done by Thomas Koenig. Using aermod.f90 from the polyhedron benchmark suite as a test case, the lseek() calls as reported by strace -c -f go roughly as - trunk before Thomas' patch: 21 million - current trunk: 5.7 million - with attached patch: 2.