[Patch, fortran] [0/11] Inline transpose part 1

Mikael Morin mikael.morin@sfr.fr
Sat Sep 11 10:21:00 GMT 2010


On Wednesday 08 September 2010 21:40:02 Janne Blomqvist 
wrote:
> On Wed, Sep 8, 2010 at 22:23, Thomas Koenig 
<tkoenig@netcologne.de> wrote:
> > Hello Mikael,
> > 
> >> The good thing:
> >> With the patches transpose(a+b) is handled without a 
temporary for (a +
> >> b).
> >> 
> >> The bad thing:
> >> With the patches function(transpose(a)) needs a temporary 
for
> >> transpose(a) which was not needed before. This is due to the 
trans stage
> >> trick which is gone, and will have to come back save us in
> >> gfc_conv_expr_descriptor: this is the point of the inline 
transpose part
> >> 2 patch serie (to come).
> > 
> > The patchset is OK.  Thanks a lot for the patches!
> > 
> > Please do the second part before 4.6 comes out to avoid a 
regression.
> > 
> > Also, I would recommend removing the transpose* functions 
from the
> > fortran-dev branch once these patches have been committed 
there.
> > 
> > Would it make sense to have a PR for both cases, to make sure 
they
> > aren't forgotten?
> > 
> > Best regards
> > 
> >        Thomas
> 
> If, as a result of these patches, the transpose implementation in
> libgfortran is no longer needed, please add a note to
> 
> http://gcc.gnu.org/wiki/LibgfortranAbiCleanup
> 
> so that we remember to delete it once we bump the ABI.

Hum, does one need some sort of registration to do so ?



More information about the Fortran mailing list