[m-rev.] for review: MLDS back-end: hoist static constants

Tyson Dowd trd at cs.mu.OZ.AU
Mon Jul 9 20:59:22 AEST 2001


On 09-Jul-2001, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> Estimated hours taken: 3
> Branches: main
> 
> Change the MLDS back-end so that it hoists *all* static constants out
> to the top level.
> 
> This is required for the IL and Java targets since they don't support
> function-local static variables.  It doesn't cost anything for the
> C or GCC targets, so for consistency we may as well do it for all targets.
> 
> Also some related bug fixes in the IL back-end.
> With these changes, string switches now work.

This is great, thanks Fergus.

Obviously it is a bit hacky to hardcode private_builtin as a special
case like this, however I think the correct solution is to not generate
references to global variables such as MR_SECTAG_LOCAL  or MR_dummy_var 
as mlds__var(...) rvals.

-- 
       Tyson Dowd           # 
                            #  Surreal humour isn't everyone's cup of fur.
     trd at cs.mu.oz.au        # 
http://www.cs.mu.oz.au/~trd #
--------------------------------------------------------------------------
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