[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