[m-rev.] for review: deleting code_exprn

Zoltan Somogyi zs at cs.mu.OZ.AU
Mon Oct 14 20:38:27 AEST 2002


On 14-Oct-2002, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> Ah, now I remember which features of the lazy LLDS code generator are not
> supported in the new LLDS code generator:
> 	(1) --no-static-ground-terms
> 	(2) the `jump' and `fast' grades.
> 
> Do we really want to get rid of those?
> 
> Note that (1) is currently needed for structure reuse,
> e.g. the tests in tests/general/structure_reuse.

About (1):

Although I see that tests/general/structure_reuse/Mmakefile specifies
--no-static-ground-terms, it doesn't specify --lazy-code, so handle_options.m
would have turned --static-ground-terms back on again. After my change,
--lazy-code isn't mentioned anywhere except in a couple of comments,
which I have now deleted.

var_locn.m already has a predicate for creating a cell dynamically.
Exporting it to code_info and making code_info export it to other modules
would take about five minutes; I can do it in this diff. If we had a pass
that marked unifications as must_be_dynamic, it would be trivial to modify
unify_gen.m to use that version of the predicate instead of the one that
checks for groundness first.

About (2), I don't know of anything that would make code_exprn and var_locn
differ in their support for the jump and fast grades. All the magic for
initializing the globals associated with labels is done behind a layer of
abstraction provided by macros; the code generators don't look behind
the curtain.

In short, this diff doesn't lose functionality on either point.

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
--------------------------------------------------------------------------



More information about the reviews mailing list