Re: memory explosion in ifcvt.c

[Richard Henderson <rth at redhat dot com>]

On Thu, Mar 29, 2007 at 12:23:11AM +0200, Steven Bosscher wrote:
> 	* ifcvt.c (struct noce_if_info): Add then_else_reversed field.
> 	(noce_get_alt_condition): Look at it to determine whether to
> 	reverse the condition or not.
> 	(noce_get_condition): Substitute the truth for lies.
> 	(noce_find_if_block): Set the then_else_reversed field.

Ok, except,

> +  /* True it this if block is not canonical.  In the canonical form of
            ^if

> +     if blocks, the THEN_BB is the block reached via the fallthru edge
> +     from TEST_BB.  For the noce transformations, we allow the symmetric
> +     form as well.  */
> +  bool then_else_reversed;

And please place this field at the end of the structure.
I know there's only one non-pointer field now, but that
might change.


r~