Re: [Patch, fortran] PR69385 - [6 regression] ICE on valid with -fcheck=mem

[Paul Richard Thomas]

Dear Janus,

Thanks - committed as revision 232850.

I hope that this signals your return - we need your help!

Paul

On 25 January 2016 at 22:13, Janus Weil  wrote:
> Hi Paul,
>
> seems we were pretty well-synchronized in posting this (in the PR it
> sounded as if you wanted me to submit it ...)
>
> In any case, the patch is ok for my taste.
>
> Thanks!
>
> Cheers,
> Janus
>
>
>
> 2016-01-25 22:02 GMT+01:00 Paul Richard Thomas 
> :
>> Dear All,
>>
>> The initial report concerns initialization assignments that should be
>> excluded from the check for assignment of scalars to unallocated
>> arrays. This part is so trivial that it does not require a test. On
>> the other hand, the block that implemented the check was plain and
>> simple wrong and the rest of the patch corrects this. It is commented
>> such as to be fully comprehensible.
>>
>> Bootstrapped and regtested on FC21/x86_64 - OK for trunk and for
>> 5-branch when all the wrinkles (PR69422 and 69423) are sorted out?
>>
>> Cheers
>>
>> Paul
>>
>> 2016-01-25  Paul Thomas  
>>
>> PR fortran/69385
>> * trans-expr.c (gfc_trans_assignment_1): Exclude initialization
>> assignments from check on assignment of scalars to unassigned
>> arrays and correct wrong code within the corresponding block.
>>
>> 2015-01-25  Paul Thomas  
>>
>> PR fortran/69385
>> * gfortran.dg/allocate_error_6.f90: New test.



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

Albert Einstein

Re: [Patch, fortran] PR69385 - [6 regression] ICE on valid with -fcheck=mem

[Paul Richard Thomas]

Hi Janus,

Thanks!

In fact I was asking you if you would submit/commit in advance of my
return :-) I'll do the business tonight.

Cheers

Paul

On 25 January 2016 at 22:13, Janus Weil  wrote:
> Hi Paul,
>
> seems we were pretty well-synchronized in posting this (in the PR it
> sounded as if you wanted me to submit it ...)
>
> In any case, the patch is ok for my taste.
>
> Thanks!
>
> Cheers,
> Janus
>
>
>
> 2016-01-25 22:02 GMT+01:00 Paul Richard Thomas 
> :
>> Dear All,
>>
>> The initial report concerns initialization assignments that should be
>> excluded from the check for assignment of scalars to unallocated
>> arrays. This part is so trivial that it does not require a test. On
>> the other hand, the block that implemented the check was plain and
>> simple wrong and the rest of the patch corrects this. It is commented
>> such as to be fully comprehensible.
>>
>> Bootstrapped and regtested on FC21/x86_64 - OK for trunk and for
>> 5-branch when all the wrinkles (PR69422 and 69423) are sorted out?
>>
>> Cheers
>>
>> Paul
>>
>> 2016-01-25  Paul Thomas  
>>
>> PR fortran/69385
>> * trans-expr.c (gfc_trans_assignment_1): Exclude initialization
>> assignments from check on assignment of scalars to unassigned
>> arrays and correct wrong code within the corresponding block.
>>
>> 2015-01-25  Paul Thomas  
>>
>> PR fortran/69385
>> * gfortran.dg/allocate_error_6.f90: New test.



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

Albert Einstein

Re: [Patch, fortran] PR69385 - [6 regression] ICE on valid with -fcheck=mem

[Janus Weil]

Hi Paul,

seems we were pretty well-synchronized in posting this (in the PR it
sounded as if you wanted me to submit it ...)

In any case, the patch is ok for my taste.

Thanks!

Cheers,
Janus



2016-01-25 22:02 GMT+01:00 Paul Richard Thomas :
> Dear All,
>
> The initial report concerns initialization assignments that should be
> excluded from the check for assignment of scalars to unallocated
> arrays. This part is so trivial that it does not require a test. On
> the other hand, the block that implemented the check was plain and
> simple wrong and the rest of the patch corrects this. It is commented
> such as to be fully comprehensible.
>
> Bootstrapped and regtested on FC21/x86_64 - OK for trunk and for
> 5-branch when all the wrinkles (PR69422 and 69423) are sorted out?
>
> Cheers
>
> Paul
>
> 2016-01-25  Paul Thomas  
>
> PR fortran/69385
> * trans-expr.c (gfc_trans_assignment_1): Exclude initialization
> assignments from check on assignment of scalars to unassigned
> arrays and correct wrong code within the corresponding block.
>
> 2015-01-25  Paul Thomas  
>
> PR fortran/69385
> * gfortran.dg/allocate_error_6.f90: New test.