[m-rev.] diff: rationalization of library/Mmakefile for IL debugging

Peter Ross pro at missioncriticalit.com
Fri Nov 22 21:24:37 AEDT 2002


Hi,


===================================================================


Estimated hours taken: 0.25
Branches: main

Clean up the Mmakefile for generating IL assemblies with debugging.

library/Mmakefile:
	Remove DEBUG_* flags because it is easier to add the flags to
	Mmake.params as documented elsewhere in the Mmakefile.
	Move munge_dll_debug_info to outside the rules which are only
	defined when LIBRARY_INTERMODULE=yes.

Index: Mmakefile
===================================================================
RCS file: /home/mercury1/repository/mercury/library/Mmakefile,v
retrieving revision 1.100
diff -u -r1.100 Mmakefile
--- Mmakefile	2 Nov 2002 06:10:12 -0000	1.100
+++ Mmakefile	22 Nov 2002 10:19:44 -0000
@@ -203,22 +203,14 @@
 # We have to hardcode the names of the runtime DLL files.
 RUNTIME_DLLS=mercury_mcpp.dll mercury_il.dll
 
-# Turn this on if you wish to enable .NET debugging.
-# But see the comments below about the install_debug_library target.
-DEBUG_MS_ILASMFLAGS=
-#DEBUG_MS_ILASMFLAGS=/debug
-#DEBUG_MS_CLFLAGS=/Zi
-#DEBUG_MS_CSCFLAGS=/debug
-
 	# -AI sets the assembly search path (just like -I for assemblies)
-MS_CLFLAGS  =	-AI`$(FIX_PATH_FOR_CL) $(RUNTIME_DIR)` -I`$(FIX_PATH_FOR_CL) $(RUNTIME_DIR)` $(DEBUG_MS_CLFLAGS)
+MS_CLFLAGS  =	-AI`$(FIX_PATH_FOR_CL) $(RUNTIME_DIR)` -I`$(FIX_PATH_FOR_CL) $(RUNTIME_DIR)`
 # For some reason, using /clr:noAssembly results in a link error when
 # building io__cpp_code.dll, because it doesn't link in the native library
 # needed for putenv().  So we need to use /link /noassembly instead.
 # MS_CL_NOASM=:noAssembly
 MS_CL_LIBS=/link /noassembly
-MS_CSCFLAGS=/t:module $(DEBUG_MS_CSCFLAGS)
-MS_ILASMFLAGS=$(DEBUG_MS_ILASMFLAGS)
+MS_CSCFLAGS=/t:module
 
 # If you do generate a new strong name, you had better update
 # compiler/mlds_to_il.m to generate references to it.  It is also hard-coded
@@ -396,15 +388,6 @@
 	$(MMAKE) $(MMAKEFLAGS) munge_dll_debug_info
 	$(MMAKE) $(MMAKEFLAGS) install_library
 
-.PHONY: munge_dll_debug_info
-munge_dll_debug_info:
-	for file in *csharp_code.dll; do \
-		ildasm /out:$$file.ild $$file; \
-		sed  's/01 00 00 01/01 00 01 01/' $$file.ild \
-			> $$file.ild.munged; \
-		ilasm /quiet /dll /debug /out:$$file $$file.ild.munged; \
-	done
-
 # The GAC is the global assembly cache, which is the system managed
 # place to put .NET assemblies.
 # gacutil installs (and uninstalls) DLLs from the GAC.
@@ -472,5 +455,14 @@
 	exit 1
 
 endif
+
+.PHONY: munge_dll_debug_info
+munge_dll_debug_info:
+	for file in *csharp_code.dll; do \
+		ildasm /out:$$file.ild $$file; \
+		sed  's/01 00 00 01/01 00 01 01/' $$file.ild \
+			> $$file.ild.munged; \
+		ilasm /quiet /dll /debug /out:$$file $$file.ild.munged; \
+	done
 
 #-----------------------------------------------------------------------------#

--------------------------------------------------------------------------
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