[m-dev.] for review: The .NET MSIL backend.

[Fergus Henderson]

On 22-Sep-2000, Tyson Dowd <trd at cs.mu.OZ.AU> wrote:
> diff -u -r1.290 options.m
> @@ -2096,7 +2100,10 @@
>  		"--fact-table-hash-percent-full <percentage>",
>  		"\tSpecify how full the `:- pragma fact_table' hash tables",
>  		"\tshould be allowed to get.  Given as an integer percentage",
> -		"\t(valid range: 1 to 100, default: 90)."
> +		"\t(valid range: 1 to 100, default: 90).",
> +
> +		"--target-debug",
> +		"\tEnable debugging of the generated target code."

The option should be documented in doc/user_guide.texi too.

Probably the best place to document this is just before --low-level-debug,
rather than there.

It seems odd to document both --c-debug and --target-debug without
saying that they are the same.  I think it would be better to document
them together:

		"--c-debug",
		"--target-debug",
		"\tEnable debugging of the generated target code."
		"\tIf the target language is C, this has the same effect as",
		"\t`--cflags ""-g"" --link-flags ""--no-strip""'.)",
		"\tIf the target language is IL, this causes the compiler to",
		"\tpass `/debug' to the IL assembler.)",

Also there should be some comments about the --debug-il-asm option,
explaining how it differs from --target-debug and why it is not
documented.

Apart from that, this change looks fine.
I'd like to see a relative diff for any changes that you make.

-- 
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"
PGP: finger fjh at 128.250.37.3        |     -- 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
--------------------------------------------------------------------------