This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: [Patch, fortran] PR42783 - [4.5 Regression] Bogus Array bounds violation with optional array argument
- From: Tobias Burnus <burnus at net-b dot de>
- To: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, fortran at gcc dot gnu dot org
- Date: Tue, 19 Jan 2010 10:33:25 +0100
- Subject: Re: [Patch, fortran] PR42783 - [4.5 Regression] Bogus Array bounds violation with optional array argument
- References: <339c37f21001182154q4316651es3972225758d09c81@mail.gmail.com>
On 01/19/2010 06:54 AM, Paul Richard Thomas wrote:
> I am not sure whether or not I have fixed PR42772 but I have certainly
> prevented the trivial problem. Perhaps Manfred could let us know,
> please?
>
Looking at your comment in the PR, it seemingly fixed that test case as
well.
> Bootstrapped and regtested FC9/x86_64 - OK for trunk?
>
OK and thanks for fixing the two regressions; together with your fix for
PR42736 (pending review) and assuming that PR 42742 is really fixed
(seems so), there are no remaining regressions :-)
Tobias
> 2010-01-19 Paul Thomas <pault@gcc.gnu.org>
>
> PR fortran/42783
> * trans-decl.c (add_argument_checking): Do not use the backend
> decl directly to test for the presence of an optional dummy
> argument. Use gfc_conv_expr_present, remembering to set the
> symbol referenced.
>
> PR fortran/42772
> * trans-decl.c (gfc_generate_function_code): Small white space
> changes. If 'recurcheckvar' is NULL do not try to reset it.
>
> 2010-01-19 Paul Thomas <pault@gcc.gnu.org>
>
> PR fortran/42783
> * gfortran.dg/bounds_check_15.f90 : New test.
>