Hi,

On Fri, 2 Dec 2011, William J. Schmidt wrote:

> > on top of your current version.  That ought to remove the added PHI 
> > expressions (and only them) from the hash table but retain the 
> > information of equality in the const_or_copies_stack.  Checking the BB 
> > wouldn't be required then.
> 
> Bootstrapped and regression tested on powerpc64-linux.

Another nit (sorry I didn't see this before :-/) :

> +      expr->ops.phi.args = (tree *) xcalloc (nargs, sizeof (tree));

This leaks, because you missed to add freeing to free_expr_hash_elt.
Apart from that looks good to me, but I can't approve.


Ciao,
Michael.

Reply via email to