Re: [Patch, fortran] PR69423 - [6 Regression] Invalid optimization with deferred-length character

[Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>]

Dear Dominique, dear all,

Many thanks for picking up the regression, which turned out to have a
trivial cause. I have taken the liberty of assuming that this is
tantamount to approval and have committed the patch as revision
233589. Any style or other wrinkles can be corrected later.

The reason that I took this liberty is that I must get past the
deferred character length business as rapidly as possible. I will
prepare a composite patch for 5-branch, with a view to committing it
in a weeks time, so that both branches have the same capability. I
have several other patches that should be backported to 5-branch,
which will take most of the time that I can devote to gfortran
tomorrow. More importantly, now that it has happened in the field, I
must fix the collisions in SELECT TYPE. The only way that I know to do
this reliably is to drop the use of a has and to use the extended type
names directly. This will take a bit of work!

Cheers

Paul


On 20 February 2016 at 13:11, Dominique d'HumiÃres <dominiq@lps.ens.fr> wrote:
>
>> Le 20 fÃvr. 2016 Ã 10:58, Paul Richard Thomas <paul.richard.thomas@gmail.com> a Ãcrit :
>>
>> Dear Dominique,
>>
>> Thanks for giving it a try. Although I cannot reproduce the problem, I
>> am pretty sure that it is due to the omission of a couple of
>> gfc_start_blocks. In the circumstances, it is a bit odd that the ICE
>> does not show up on my system!
>>
>> Please give the attached a whirl.
>
> It fixes the PR without regression.
>
> Thanks,
>
> Dominique
>
>



-- 
The difference between genius and stupidity is; genius has its limits.

Albert Einstein