[m-rev.] for review: drop support for lcc
Peter Wang
novalazy at gmail.com
Mon Sep 29 16:55:37 AEST 2014
On Mon, 29 Sep 2014 15:52:00 +1000 (EST), Julien Fischer <jfischer at opturion.com> wrote:
>
> For review by anyone.
>
> ---------------------
>
> Drop support for lcc.
>
> Drop support for lcc as a C compiler. lcc itself does not seem to have been
> updated since around 2010 and Mercury's support for it has not been updated for
> quite a while before that.
>
> Derivatives of lcc, for example lcc-win32, would almost certainly need to be
> treated separately anyway, so this change doesn't affect them (i.e. they are
> already not supported).
>
> configure.ac:
> Do not recognise lcc as a C compiler.
>
> Unrelated change: remove residual support for a.out executables on Linux.
>
> scripts/mgnuc_file_opts.sh-subr:
> Do not define the macros __EXTENSIONS__ and _GNU_SOURCE if the mgnuc script
> is invoked with --no-ansi. This was originally done to support lcc on Linux.
> Doing so on *all* systems as this code actually does is not a good idea.
> (If it becomes necessary to reinstate this behaviour on some system, it
> should be controlled from the configure script not here.)
>
> scripts/mgnuc.in:
> scripts/ml.in:
> compiler/compile_target_code.m:
> compiler/handle_options.m:
> compiler/globals.m:
> Delete support for lcc.
>
> README.lcc:
> Delete this file.
It looks fine. git grep 'lcc' reveals a few more mentions and
workarounds that could be removed.
Peter
More information about the reviews
mailing list