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

Thomas Koenig tkoenig@netcologne.de
Wed Sep 8 19:40:00 GMT 2010


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



More information about the Fortran mailing list