Re: [tree-ssa] Initial formal temporary table PATCH, not applied

[Diego Novillo <dnovillo at redhat dot com>]

On Mon, 2003-04-28 at 00:01, Jason Merrill wrote:

> With these changes, the patch speeds up compilation of insn-attrtab.c on my
> laptop by about 5%, reduces the string pool size by about 15%, and reduces
> the "total" line in -fmem-report from 30M to 29M.  This is all with a
> stage1 compiler.
> 
Cool.

> Morgan also talks about deleting redundant computations while building the
> flow graph (i.e. during gimplification), but I suspect that the payoff
> isn't worth the headache of tracking whether operands have been modified,
> since it's so easy to do it in the renamer.
> 
Agreed.

> tree-ssa.c patch attached below.  Andrew, was the "ops1 == ops2" bit a
> workaround for some bug, or is it indeed a typo?  I suppose I'll find out
> when I try to bootstrap this.
> 
That was my change.  For the life of me I can't remember why I added
that test.  It certainly seems unnecessary.


Diego.