Did someone actually approve this patch? Apparently it was committed
as r262744 and caused the following regression:

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86543

Cheers,
Janus



2018-06-27 23:07 GMT+02:00 Fritz Reese <fritzore...@gmail.com>:
> The attached patch fixes PR fortran/83184, which is actually two
> distinct bugs as described in the PR. Passes regtests.
>
> The patch is attached. OK for trunk and 7/8-branch?
>
> From 238f0a0e80c93209bb4e62ba2f719f74f5da164f Mon Sep 17 00:00:00 2001
> From: Fritz Reese <fritzore...@gmail.com>
> Date: Wed, 27 Jun 2018 16:16:31 -0400
> Subject: [PATCH 2/3] PR fortran/83184
>
> Fix handling of invalid assumed-shape/size arrays in legacy initializer
> lists.
>
> gcc/fortran/
>
>         * decl.c (match_old_style_init): Initialize locus of variable expr 
> when
>         creating a data variable.
>         (match_clist_expr): Verify array is explicit shape/size before
>         attempting to allocate constant array constructor.
>
> gcc/testsuite/
>
>         * gfortran.dg/assumed_rank_14.f90: New testcase.
>         * gfortran.dg/assumed_rank_15.f90: New testcase.
>         * gfortran.dg/dec_structure_8.f90: Update error messages.
>         * gfortran.dg/dec_structure_23.f90: Update error messages.
> ---
>  gcc/fortran/decl.c                             | 63 
> +++++++++++++++-----------
>  gcc/testsuite/gfortran.dg/assumed_rank_14.f90  | 11 +++++
>  gcc/testsuite/gfortran.dg/assumed_rank_15.f90  | 11 +++++
>  gcc/testsuite/gfortran.dg/dec_structure_23.f90 |  6 +--
>  gcc/testsuite/gfortran.dg/dec_structure_8.f90  |  6 +--
>  5 files changed, 64 insertions(+), 33 deletions(-)
>  create mode 100644 gcc/testsuite/gfortran.dg/assumed_rank_14.f90
>  create mode 100644 gcc/testsuite/gfortran.dg/assumed_rank_15.f90

Reply via email to