[m-rev.] for review: move analysis framework into compiler

Julien Fischer juliensf at csse.unimelb.edu.au
Sun Feb 17 13:47:22 AEDT 2008


On Fri, 15 Feb 2008, Peter Wang wrote:

> Estimated hours taken: 1.5
> Branches: main
>
> Move the intermodule analysis framework into the `compiler' directory, in
> preparation for making it specific to the Mercury compiler, rather than having
> it generic in case some fictional being might want to use it with a .NET
> compiler one day.  This will make it easier to use and modify.

For the record, I have discussed this with Peter in person and I agree
with this change.

> analysis/analysis.file.m:
> analysis/analysis.m:
> 	Move to the `compiler' directory.
>
> analysis/README:
> 	Mention that the code has been moved.
>
> analysis/.cvsignore:
> analysis/.mgnuc_copts:
> analysis/.mgnuc_opts:
> analysis/.nocopyright:
> analysis/ANALYSIS_FLAGS.in:
> analysis/Mercury.options:
> analysis/Mmakefile:
> analysis/mer_analysis.m:
> 	Delete these files.
>
> compiler/top_level.m:
> 	Include analysis.m as a new package.
>
> compiler/Mercury.options:
> 	Add a bug workaround line from analysis/Mercury.options.
>
> compiler/notes/compiler_design.html:
> compiler/notes/overall_design.html:
> 	Update documentation.
>
> Mmake.workspace:
> Mmakefile:
> configure.in:
> compiler/.mgnuc_copts:
> compiler/COMP_FLAGS.in:
> compiler/Mmakefile:
> deep_profiler/.mgnuc_copts:
> scripts/Mmake.vars.in:
> scripts/c2init.in:
> scripts/mercury_config.in:
> scripts/prepare_tmp_dir_fixed_part.in:
> tools/binary:
> tools/binary_step:
> tools/bootcheck:
> tools/lmc.in:
> tools/make_arena:
> compiler/notes/coding_standards.html:
> 	Remove references to analysis directory and libmer_analysis.
>
>
> I would prefer the analysis*.m files to retain their history across the
> move, but it's not critical.

Did you consider leaving them in the present location and either using
mmc -f, or just copying them across to the compiler directory at
compile time?

Please wait a few days before committing in case anyone else wishes
to comment.

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