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

[Mikael Morin]

Le 11.09.2010 12:22, Richard Guenther a écrit :
>
> 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.
>
Thanks. Done.
I was really scratching my head on this.

Mikael