[Patch, fortran] PR92976 - [8/9/10 Regression][OOP] ICE in trans_associate_var, at fortran/trans-stmt.c:1963

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[Patch, fortran] PR92976 - [8/9/10 Regression][OOP] ICE in trans_associate_var, at fortran/trans-stmt.c:1963

Paul Richard Thomas
I am a tiny bit skeptical that this is a regression but I will check.
However, it has clearly been there from the early days of OOP without
being picked up.

The fix is to ensure that the temporary has the correct type of array spec.

Regtested on x86_64/FC31 - OK for trunk and 8-/9- branches ?

Cheers

Paul

2020-02-29  Paul Thomas  <[hidden email]>

    PR fortran/92976
    * match.c (select_type_set_tmp): If the selector array spec has
    explicit bounds, make the temporary's bounds deferred.

2020-02-29  Paul Thomas  <[hidden email]>

    PR fortran/92976
    * gfortran.dg/select_type_48.f90 : New test.

submit.diff (2K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [Patch, fortran] PR92976 - [8/9/10 Regression][OOP] ICE in trans_associate_var, at fortran/trans-stmt.c:1963

Thomas Koenig-6
Hi Paul,

> Regtested on x86_64/FC31 - OK for trunk and 8-/9- branches ?

OK, and thanks for the patch.

I think it makes sense to get this into gcc 9.3, which would need
a backport before the 5th of March.

Regards

        Thomas
Reply | Threaded
Open this post in threaded view
|

Re: [Patch, fortran] PR92976 - [8/9/10 Regression][OOP] ICE in trans_associate_var, at fortran/trans-stmt.c:1963

Paul Richard Thomas
Committed to head as r10-6954-g957a1b14e99596610abb0777ca86a1c80dde24e0.

Thanks, Thomas

Paul

On Sun, 1 Mar 2020 at 13:43, Thomas Koenig <[hidden email]> wrote:

>
> Hi Paul,
>
> > Regtested on x86_64/FC31 - OK for trunk and 8-/9- branches ?
>
> OK, and thanks for the patch.
>
> I think it makes sense to get this into gcc 9.3, which would need
> a backport before the 5th of March.
>
> Regards
>
>         Thomas



--
"If you can't explain it simply, you don't understand it well enough"
- Albert Einstein