This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Fix PR tree-opt/32527: ICE in build2_stat (aff_combination_scale)

From: "Andrew Pinski" <pinskia at gmail dot com>
To: "Ian Lance Taylor" <iant at google dot com>
Cc: Andrew_Pinski at playstation dot sony dot com, gcc-patches at gcc dot gnu dot org
Date: Sat, 30 Jun 2007 01:47:34 -0700
Subject: Re: [PATCH] Fix PR tree-opt/32527: ICE in build2_stat (aff_combination_scale)
References: <OF6F92DFDB.275BED40-ON8825730A.00089BDD-8825730A.000BA0F7@playstation.sony.com> <m3fy4am7b7.fsf@localhost.localdomain>

On 29 Jun 2007 20:48:12 -0700, Ian Lance Taylor <iant@google.com> wrote:

This looks incomplete to me.  The comment in tree-affine.h says that
the type of comb->rest must be comb->type, but this change breaks that
invariant.  Is the comment wrong?


This comment is wrong since the addition of POINTER_PLUS_EXPR.  Though
it looks like there is a mistake in the whole tree-addine.c around the
type of rest.

I will look into fixing this more completely tomorrow.  I think I
might have a patch already which fixes up tree-affine.c correctly at
work but I forgot to look at it before looking into this bug.

The reason why we hit this bug is because rest is not null and that is
because we go over MAX_AFF_ELTS (8).

Thanks,
Andrew Pinski

References:
- [PATCH] Fix PR tree-opt/32527: ICE in build2_stat (aff_combination_scale)
  - From: Andrew_Pinski
- Re: [PATCH] Fix PR tree-opt/32527: ICE in build2_stat (aff_combination_scale)
  - From: Ian Lance Taylor

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]