On 03/26/2012 09:14 AM, Meador Inge wrote:

> Hi All,
> This patch fixes an ICE that occurs when attempting to fold nested 
> trees that have conversions in between the indirect references.  For example:
> constexpr unsigned long b = *((ul_ptr)(*((ul_ptr)0x0)));
> What happens is that 'cxx_fold_indirect_ref' gets the top indirect reference,
> strips out all the conversions ending up with the nested indirect reference,
> and then asserts that the type is a pointer type, which (obviously) it isn't.
> This patch fixes the problem by exiting the fold for non-pointer sub-trees
> instead of doing the assert.  'fold_indirect_ref_1' already does things this
> way.
> Tested on x86_64 GNU/Linux.  OK?
> Also, This fixes the problem on trunk, but I have reproduced the issue back to
> the 4.6 branch as well.  On the 4.6 branch I think it is OK to just remove the
> 'gcc_assert (POINTER_TYPE_P (subtype))' assert.  I tested this and saw no
> regressions.
> P.S.  If it is OK can some please commit for me?  I don't have write access.

Hi Jason,

I saw where the fix for this got committed in r185890.  Thanks for applying
it for me.  What about the release branches?  I can reproduce the ICE in 4.6
and 4.7.  The patch from trunk applies to 4.7.  For 4.6 I think we can just
drop the assert.

Meador Inge
CodeSourcery / Mentor Embedded

Reply via email to