[m-rev.] for review: write out user sharing annotations to .opt files
Julien Fischer
juliensf at csse.unimelb.edu.au
Mon Mar 3 20:06:23 AEDT 2008
On Mon, 3 Mar 2008, Peter Wang wrote:
> This would cause a mutual dependency between mercury_to_mercury.m and
> prog_data.m. Maybe attributes_to_strings should be in
> mercury_to_mercury.m?
Based on a quick grep through the compiler directory, mercury_to_mercury
seems to be the only place that calls it anyway, so move it there.
(It should be renamed to something like
foreign_code_attributes_to_strings as there are few other declarations
that have attributes, e.g. pragma foreign_{decl, type, export_enum}, mutable.)
> Estimated hours taken: 2.5
> Branches: main
>
> Write out user-specified structure sharing annotations
> (no_sharing/unknown_sharing/sharing) on foreign_procs, e.g. when making `.opt'
> files.
>
> compiler/prog_data.m:
> Make `attributes_to_strings' take into account user sharing
> annotations.
>
> compiler/mercury_to_mercury.m:
> Conform to the change.
>
> tests/valid/Mercury.options:
> tests/valid/Mmakefile:
> tests/valid/intermod_user_sharing.m:
> tests/valid/intermod_user_sharing_2.m:
> Add a test case.
The diff is otherwise 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