[m-rev.] for review: pragma foreign_import_module on the IL backend

Peter Ross pro at missioncriticalit.com
Mon Dec 23 22:16:35 AEDT 2002


petdr wrote:
> I am just writing the changes to the documentation now.
>
> diff log file:
> --- zzlog.foreign_import_module 2002-12-20 14:31:52.000000000 +0100
> +++ zzlog.foreign_import_module3 2002-12-20 14:45:11.000000000 +0100
> @@ -1,5 +1,5 @@
>
> -Estimated hours taken: 8
> +Estimated hours taken: 12
>  Branches: main
>
>  Get pragma foreign_import_module working on the IL backend.
> @@ -17,6 +17,10 @@
>   For managed C++ output a #using for every module implied by the
>   pragma foreign_import_module.
>
> +compiler/compile_target_code.m:
> + Add the assemblies implied by the pragma foreign_import_module
> + to the C# compiler's command line.
> +
>  compiler/modules.m:
>   Factor the code for outputting the dependencies implied by the
>   pragma foreign_import_modules so that they work for both the
> @@ -24,6 +28,16 @@
>   For C# add a reference to every module implied by the pragma
>   foreign_import_module.
>
> +compiler/make.dependencies.m:
> + Add the dependency that before we can build a MC++ or C#
> + assembly all the assemblies implied by the
> + foreign_import_modules must be built.
> +
> +compiler/make.module_target.m:
> + Detect when circular dependencies caused by cyclic
> + pragma foreign_import_module on the IL backend occur and print
> + an informative error message.
> +
>  compiler/modules.m:
>  compiler/prog_io_pragma.m:
>   Remove constraints the pragma foreign_import_modules can only
>
With these changes, is it alright to check this change in after I add the
s/error/Error/ suggested by fjh?




---
Outgoing mail is certified Virus Free.
Checked by AVG anti-virus system (http://www.grisoft.com).
Version: 6.0.419 / Virus Database: 235 - Release Date: 11/13/2002

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