Re: [patch, fortran] Unformatted array IO performance improvement

[Janne Blomqvist <jblomqvi at cc dot hut dot fi>]

On Sun, Sep 25, 2005 at 09:55:34PM +0200, Thomas Koenig wrote:
> On Sun, Sep 25, 2005 at 09:26:45PM +0300, Janne Blomqvist wrote:
> > :ADDPATCH fortran:
> > 
> > Ok, here is yet another updated version of the patch. The most
> > significant difference compared to previous versions is that the
> > transfer_array library entry point now has a second argument 'charlen'
> > which specifies the length of the characters in case the type is
> > character. For non-character types the frontend just generates a
> > constant which is not used. This extra argument is needed because the
> > charlen which is stored in the descriptor is clobbered if charlen >
> > 2**24. It also makes it possible to support non-8-bit charset later on
> > if we wish.
> 
> This is OK for mainline.

Great! Could someone commit it, as I don't have commit privileges?

> This should pretty much lay PR 16339 and 21820
> to rest (maybe you could mention them in the ChangeLog).

I guess you mean 16339 and 23363? 

21820 uses implied do loops, so doesn't use transfer_array(). 21820
was significantly improved by my recent already committed patch that
got rid of mmap. We still lose to ifort though, so I don't know if it
should be kept open or not.

-- 
Janne Blomqvist