[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