[m-rev.] for review: fix .NET compiler command lines

[Fergus Henderson]

On 13-Nov-2002, Simon Taylor <stayl at cs.mu.OZ.AU> wrote:
> 
> compiler/compile_target_code.m:
> 	Apply a patch to the MSVC command line for compiling
> 	Managed C++, which Fergus previous applied in Mmake.rules -
> 	s/-link -noentry mscoree.lib -dll/\/LD/

I had the same change lying around in one of my workspaces,
but had not yet gotten around to posting or committing it.

> @@ -246,7 +246,7 @@
>  	{ join_string_list(CSCFlagsList, "", "", " ", CSCFlags) },
>  	globals__io_lookup_bool_option(target_debug, Debug),
>  	{ Debug = yes ->
> -		DebugOpt = "" % XXX
> +		DebugOpt = "/debug+ /debug:full"

That's missing a trailing space, isn't it?
Oh, I see below that there is an extra space before
the thing which follows DebugOpt.  Better to remove
that and put the trailing space here.

Also, have you tested this?  There seem to be problems mixing C# files
compiled with debugging with IL or MC++ files compiled with debugging in
the same assembly.  Maybe this is OK, because I think only the standard
library gets put in a single assembly...  but I would not be confident
about this unless it has been tested.  If you haven't tested it on .NET,
I think it may be best to leave the C# debugging option as empty for now.

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
The University of Melbourne         |  of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh>  |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
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
--------------------------------------------------------------------------