[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