[m-rev.] For review by Fergus: Loop Invariant Hoisting

[Zoltan Somogyi]

On 16-Oct-2002, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> Also, what effect does this have on performance?  (E.g. for the compiler
> itself, or the benchmarks in the `bench' directory).  We don't want to
> enable the optimization by default if it ends up making performance worse.
> It would be a good idea to run some benchmarks to double-check that the
> intended optimization really does improve things.
> 
> How much space cost does it have?
> Should it be disabled if --optimize-space is specified?

Ralph, you can use tools/makebatch to answer these questions. I'll come over
to your desk tomorrow to show you how to use it.

> > -		DebugLiveness, PredIdInt, Goal1, VarSet, ModuleInfo, IO0, IO1),
> > +		DebugLiveness, PredIdInt, Goal1, VarSet, ModuleInfo, IO0b, IO1),
> 
> Rather than using "IO0b", please renumber the IO* variables.
> ("IO0b" is OK for a quick hack, but for the CVS repository we
> have higher standards ;-)

Don't we have some fancy new notation that avoids the need for renumbering :-)

Zoltan.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------