[m-rev.] For review: Remove support for Managed C++

Julien Fischer juliensf at csse.unimelb.edu.au
Mon Jul 9 12:32:18 AEST 2007


On Sat, 7 Jul 2007, Jonathan Morgan wrote:

> This change is fairly straightforward and boring.  What I want review
> comments on are:
> * Is there anything that I've missed removing?

Not that I can see.

> * Should this change be mentioned in NEWS?

Yes.

> * Should I move mlds_to_managed.m back to mlds_to_csharp.m (where it
> was originally), or leave it as is?

No, leave it as is, resurrecting files in CVS is problematic.

> I will be bootchecking this change in hlc.gc before committing to make
> sure, as it is the only really important MLDS-based grade.

Please bootcheck it in hl.gc as well.

> Estimated hours taken: 3
> Branches: main
>
> Removed support Managed C++ as a foreign language for the IL backend.  This 
> is
> being removed for the following reasons:
> * It is only supported by Microsoft, making it harder to use code
> using it with other CIL platforms (e.g. Mono).
> * It is not used in any of the standard library (only C# and IL are used).
> * Removing it will simplify the dependencies and the build process.
> * Microsoft have deprecated it in favour of C++/CLI.
>
> aclocal.m4:
> 	Remove checks for the Microsoft.NET C++ compiler.
>
> compiler/globals.m:
> 	Delete Managed C++ from the set of supported languages.
>
> compiler/mlds_to_managed.m:
> 	Remove support for Managed C++ code generation.
>
> compiler/*.m:
> 	Remove all references to Managed C++.
>
> runtime/Mmakefile:
> runtime/mercury_mcpp.h:
> 	Remove the Managed C++ part of the Mercury runtime.
>
> runtime/mercury_il.il:
> runtime/mercury_typeclass_info.h:
> runtime/mercury_type_info.h:
> 	Remove all references to Managed C++, including instructions to keep
> 	certain parts in sync with Managed C++ part of the runtime.
>
> doc/reference_manual.texi:
> 	Remove the Managed C++ sections in the Foreign Language Interface
> 	chapter.
>
> doc/user_guide.texi:
> 	Remove the documentation of Managed C++ related flags.
>
> library/Mmakefile:
> 	Remove Managed C++ flag settings.
>
> library/io.m:
> 	Remove commented out Managed C++ code.
>
> library/list.m:
> 	Change a comment referring to Managed C++ to refer to C# instead.
>
> scripts/Mmake.rules:
> scripts/Mmake.vars.in:
> 	Remove all rules and variables for Managed C++ from mmake.

That looks fine.

Julien.




--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list