[m-rev.] for review; compiler changes needed for il grade

Peter Ross pro at missioncriticalit.com
Mon Jul 1 19:10:31 AEST 2002


Hi,

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


Estimated hours taken: 0.5
Branches: main

Misc review changes pointed out by Fergus, for my last change to
improve compilation support for the il grade.

compiler/handle_options.m:
	Mention that intermodule optimization is required for the il
	grade so that *abstract* equivalence types can be expanded.

compiler/mlds.m:
	Fix a comment.
	
compiler/mlds_to_managed.m:
	Don't automatically import private_builtin.dll and
	builtin.dll, as they are contained in the import list.

Index: handle_options.m
===================================================================
RCS file: /home/mercury1/repository/mercury/compiler/handle_options.m,v
retrieving revision 1.141
diff -u -r1.141 handle_options.m
--- handle_options.m	28 Jun 2002 09:13:42 -0000	1.141
+++ handle_options.m	1 Jul 2002 08:57:41 -0000
@@ -330,8 +330,8 @@
 	%             So we should investigate re-enabling static ground terms.
 	%   - intermodule optimization
 	%	  This is only required for high-level data and is needed
-	%	  so that equivalence types can be expanded.  They need to be
-	%	  expanded because .NET requires that the structural
+	%	  so that abstract equivalence types can be expanded.  They
+	%	  need to be expanded because .NET requires that the structural
 	%	  representation of a type is known at all times.
 	( { Target = il } ->
 		globals__io_set_gc_method(none),
Index: mlds.m
===================================================================
RCS file: /home/mercury1/repository/mercury/compiler/mlds.m,v
retrieving revision 1.95
diff -u -r1.95 mlds.m
--- mlds.m	30 Jun 2002 17:06:30 -0000	1.95
+++ mlds.m	1 Jul 2002 08:57:42 -0000
@@ -379,7 +379,8 @@
 % MLDS package.
 :- func mlds_module_name_to_package_name(mlds_module_name) = sym_name.
 
-% Is the current module a member of the std library, is so which module is it?
+% Is the current module a member of the std library,
+% and if so which module is it?
 :- pred is_std_lib_module(mlds_module_name::in, string::out) is semidet.
 
 % Given an MLDS module name (e.g. `foo.bar'), append another class qualifier
Index: mlds_to_managed.m
===================================================================
RCS file: /home/mercury1/repository/mercury/compiler/mlds_to_managed.m,v
retrieving revision 1.3
diff -u -r1.3 mlds_to_managed.m
--- mlds_to_managed.m	28 Jun 2002 09:13:44 -0000	1.3
+++ mlds_to_managed.m	1 Jul 2002 08:57:44 -0000
@@ -177,9 +177,11 @@
 	( { mercury_std_library_module_name(ModuleName) } ->
 		io__write_strings([
 			"#using ""mercury_mcpp.dll""\n",
-			"#using ""mercury_il.dll""\n",
-			"#using ""private_builtin.dll""\n",
-			"#using ""builtin.dll""\n"])
+			"#using ""mercury_il.dll""\n"])
 	;
 		[]
 	),

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