On Thu, Aug 25, 2011 at 12:32 PM, Tom de Vries <vr...@codesourcery.com> wrote:
> Jakub,
>
> This patch fixes a segmentation violation, which occurs when printing a 
> MEM_REF
> or COMPONENT_REF containing a released ssa name.  This can happen when we 
> print
> basic blocks upon removal, enabled by -ftree-dump-tree-*-details (see
> remove_bb:tree-cfg.c).

Where do we dump stmts there?

> Bootstrapped and reg-tested on x86_64.
>
> OK for trunk?

At least

  TREE_TYPE (TREE_OPERAND (node, 1)) != NULL_TREE

is always true.

The comment before the new lines is now in the wrong place and this
check at least needs a comment as well.

But - it's broken to dump freed stuff, why and where do we do this?

Richard.

> Thanks,
> - Tom
>
> 2011-08-25  Tom de Vries  <t...@codesourcery.com>
>
>        * tree-pretty-print (dump_generic_node): Test for NULL_TREE before
>        accessing TREE_TYPE.
>

Reply via email to