This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch, fortran] Fix PR 66111
- From: Mikael Morin <mikael dot morin at sfr dot fr>
- To: Thomas Koenig <tkoenig at netcologne dot de>, "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: Tue, 12 May 2015 22:35:34 +0200
- Subject: Re: [patch, fortran] Fix PR 66111
- Authentication-results: sourceware.org; auth=none
- Authentication-results: sfrmc.priv.atos.fr; dkim=none (no signature); dkim-adsp=none (no policy) header dot from=mikael dot morin at sfr dot fr
- References: <55525D6A dot 8050106 at netcologne dot de>
Hello,
Le 12/05/2015 22:07, Thomas Koenig a Ãcrit :
> Hello world,
>
> this patch fixes a regression from the inline matmul patch by
> not inlining a case that is not yet handled, namely vector
> subscripts.
>
> Regression-tested. OK for trunk?
>
OK with...
>
> Index: frontend-passes.c
> ===================================================================
> --- frontend-passes.c (Revision 222984)
> +++ frontend-passes.c (Arbeitskopie)
> @@ -2752,6 +2771,10 @@ inline_matmul_assign (gfc_code **c, int *walk_subt
> || matrix_b->expr_type != EXPR_VARIABLE)
> return 0;
>
> + if (has_dimen_vector_ref (expr1) ||
> + has_dimen_vector_ref (matrix_a) || has_dimen_vector_ref (matrix_b))
> + return 0;
> +
...the operator || at the beginning of the line.
Thanks.
Mikael