[Patch, Fortran] PR fortran/37749 ICE on array section with vector subscript

[Paul Richard Thomas]

Mikael,

I took action last night on this and PR37903 because I was concerned
that we were going to run out of time on stage 3.  I'm sorry about
doubling up our effort.  I tried to contact you before the commit but
failed.  If you have any problem with what I committed, please get in
touch and we will put it right.  You have been attributed in the
ChangeLogs:-)

Cheers

Paul

On Thu, Oct 30, 2008 at 8:44 PM, Mikael Morin <mikael.morin@tele2.fr> wrote:
> Hello,
>
> here comes the e-mail version of my patch posted on bugzilla:
> with a ChangeLog entry and the testcase.
>
> As a reminder, this patch looks for a NULL loop->to[n] before setting
> the temporary descriptor to check whether we are using a
> callee-allocated array or not.
> This prevents setting the descriptor with a NULL size (which segfaults).
>
> I didn't take the run-time check from the original testcase, as it is
> just a compilation failure.
>
> regression tested on x86_64-unknown-linux-gnu.
>
> Mikael
>
>
> 2008-10-30  Mikael Morin  <mikael.morin@tele2.fr>
>
>        PR fortran/37749
>        * trans-array.c (gfc_trans_create_temp_array): Check whether
>        there is one NULL loop->to[n] before setting the descriptor
>        with the loop bounds.
>
> 2008-10-30  Mikael Morin  <mikael.morin@tele2.fr>
>
>        PR fortran/37749
>        * gfortran.dg/vector_subscript_4.f90: New test.
>



-- 
The knack of flying is learning how to throw yourself at the ground and miss.
       --Hitchhikers Guide to the Galaxy