This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: [Patch, fortran] PR36091
- From: Steve Kargl <sgk at troutmask dot apl dot washington dot edu>
- To: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
- Cc: "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, gcc patches <gcc-patches at gcc dot gnu dot org>
- Date: Sun, 5 Apr 2009 13:30:56 -0700
- Subject: Re: [Patch, fortran] PR36091
- References: <339c37f20904051310h385ac1cr8b82a4d9a40509be@mail.gmail.com>
On Sun, Apr 05, 2009 at 10:10:03PM +0200, Paul Richard Thomas wrote:
> The symbol generated by trans-stmt.c(forall_make_variable_temp) uses
> lbound = 0 for all dimensions. This does not matter, except for
> bounds checking. The fix is to use the array_spec and convert the
> bounds, instead of using the array descriptor. Rather than adding a
> test, forall_13.f90, where the problem was identified, has been
> modified to use the -fbounds-check option.
>
> Bootstrapped and regtested on FC9/x86_64 - OK for trunk?
>
> Paul
>
> 2009-04-05 Paul Thomas <pault@gcc.gnu.org>
>
> PR fortran/36091
> * trans-array.c (gfc_conv_array_ref): If the symbol has the
> temporary attribute use the array_spec for the bounds.
> * gfortran.h : Add the temporary field to the structure
> 'symbol_attribute'.
> * trans-stmt.c (forall_make_variable_temp): Set the symbol's
> temporary attribute.
>
> 2009-04-05 Paul Thomas <pault@gcc.gnu.org
>
> PR fortran/36091
> * gfortran.dg/forall_13.f90: Add -fbounds-check option.
OK.
--
Steve