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

[Janus Weil <janus at gcc dot gnu dot org>]

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 <paul.richard.thomas@gmail.com>:
> 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  <pault@gcc.gnu.org>
>
>     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  <pault@gcc.gnu.org>
>
>     PR fortran/69385
>     * gfortran.dg/allocate_error_6.f90: New test.