[PATCH] Fix PR middle-end/39976, 200.sixtrack degradation
William J. Schmidt
wschmidt@linux.vnet.ibm.com
Mon Dec 5 14:36:00 GMT 2011
On Mon, 2011-12-05 at 14:59 +0100, Michael Matz wrote:
> 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.
>
/facered :(
Thanks. Will fix...
Bill
>
> Ciao,
> Michael.
>
More information about the Gcc-patches
mailing list