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

Richard Guenther richard.guenther@gmail.com
Sat Sep 11 11:37:00 GMT 2010


On Sat, Sep 11, 2010 at 11:40 AM, Mikael Morin <mikael.morin@sfr.fr> wrote:
> 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 ?

Just use a WikiWord as username.

Richard.



More information about the Fortran mailing list