This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: [Patch, fortran] [0/11] Inline transpose part 1
- From: Thomas Koenig <tkoenig at netcologne dot de>
- To: Mikael Morin <mikael dot morin at sfr dot fr>
- Cc: "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 08 Sep 2010 21:23:07 +0200
- Subject: Re: [Patch, fortran] [0/11] Inline transpose part 1
- References: <4C8254A0.9020907@sfr.fr>
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