This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fix vectorizer ICE when building Mozilla
- From: Andrew Pinski <pinskia at gmail dot com>
- To: Jan Hubicka <hubicka at ucw dot cz>
- Cc: gcc-patches at gcc dot gnu dot org, rguenther at suse dot de, matz at suse dot de
- Date: Sun, 22 Apr 2012 18:30:54 -0700
- Subject: Re: Fix vectorizer ICE when building Mozilla
- References: <20120422230440.GB29034@kam.mff.cuni.cz>
On Sun, Apr 22, 2012 at 4:04 PM, Jan Hubicka <hubicka@ucw.cz> wrote:
> Hi,
> when building Mozilla we ICE in expr_invariant_in_loop_p walking ARRAY_REF, because one of the
> fields is NULL (that is allowed)
>
> Bootstrapped/regtested x86_64-linux, OK?
>
> Honza
>
> Â Â Â Â* tree-ssa-loop-ivopts.c (expr_invariant_in_loop_p): Bail out at NULL tree refs.
> Index: tree-ssa-loop-ivopts.c
> ===================================================================
> *** tree-ssa-loop-ivopts.c   Â(revision 186688)
> --- tree-ssa-loop-ivopts.c   Â(working copy)
> *************** expr_invariant_in_loop_p (struct loop *l
> *** 1411,1417 ****
>
> Â Âlen = TREE_OPERAND_LENGTH (expr);
> Â Âfor (i = 0; i < len; i++)
> ! Â Â if (!expr_invariant_in_loop_p (loop, TREE_OPERAND (expr, i)))
> Â Â Â Âreturn false;
>
> Â Âreturn true;
> --- 1411,1417 ----
>
> Â Âlen = TREE_OPERAND_LENGTH (expr);
> Â Âfor (i = 0; i < len; i++)
> ! Â Â if (TREE_OPERAND (expr, i) && !expr_invariant_in_loop_p (loop, TREE_OPERAND (expr, i)))
Long lines, but I think otherwise this is obvious.
Thanks,
Andrew Pinski
> Â Â Â Âreturn false;
>
> Â Âreturn true;