[m-dev.] for review: IL backend bugfixes
Fergus Henderson
fjh at cs.mu.OZ.AU
Sun Oct 22 16:40:45 AEDT 2000
On 22-Oct-2000, Tyson Dowd <trd at cs.mu.OZ.AU> wrote:
> diff -u -r1.13 ml_elim_nested.m
> @@ -1384,7 +1393,10 @@
> local_vars :: list(mlds__defn),
>
> % Type of the introduced environment struct
> - env_type_name :: mlds__type
> + env_type_name :: mlds__type,
> +
> + % Type of the introduced environment struct
> + env_ptr_type_name :: mlds__type
The comment for the second one is a duplicate of the
comment for the first one, which is confusing.
> +
> +:- func this_file = string.
> +this_file = "mlds_to_ilasm.m".
I suggest you add `:- end_module mlds_to_ilasm.m' here.
That way, if someone changes the name of this module, and doesn't
change this code, the compiler will warn them.
[to be continued]
--
Fergus Henderson <fjh at cs.mu.oz.au> | "I have always known that the pursuit
WWW: <http://www.cs.mu.oz.au/~fjh> | of excellence is a lethal habit"
| -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-developers mailing list
Post messages to: mercury-developers at cs.mu.oz.au
Administrative Queries: owner-mercury-developers at cs.mu.oz.au
Subscriptions: mercury-developers-request at cs.mu.oz.au
--------------------------------------------------------------------------
More information about the developers
mailing list