[m-rev.] for review: --use-atomic-cells

Julien Fischer juliensf at csse.unimelb.edu.au
Sat Aug 19 01:25:11 AEST 2006


On Fri, 18 Aug 2006, Zoltan Somogyi wrote:

> runtime/mercury_heap.h:
> 	Define atomic equivalents of the few heap allocation macros
> 	that didn't already have one. These macros are used by the LLDS
> 	backend.
>
> runtime/mercury.h:
> 	Define an atomic equivalent of the MR_new_object macro.
> 	These macros are used by the MLDS backend.
>
> 	Use MR_new_object_atomic instead of MR_new_object to box floats.
>
> compiler/hlds_data.m:
> compiler/llds.m:
> compiler/mlds.m:
> 	Modify the representations of the heap allocations constructs
> 	to include a flag that says whether we should use the atomic variants
> 	of the heap allocation macros.
>
> compiler/llds_out.m:
> compiler/mlds_to_c.m:
> 	Respect this extract flag when emitting C code.
>
> 	In mlds_to_c.m, also add some white space that makes the code easier
> 	for humans to read.
>
> compiler/type_util.m:
> 	Add a mechanism for finding out whether we can put a value of a given
> 	type into an atomic cell.
>
> 	Put the definitions of functions and predicates in this module
> 	in the same order as their declarations.
>
> 	Turn some predicates into functions. Change the argument order of
> 	some predicates to conform to our usual conventions.
>
> compiler/unify_gen.m:
> compiler/ml_unify_gen.m:
> 	Use the new mechanism in type_util.m to generate code that creates
> 	atomic heap cells if this is possible and is requested.
>
> compiler/code_info.m:
> compiler/var_locn.m:
> 	Act on the information provided by unify_gen.m.
>
> compiler/options.m:
> doc/user_guide.texi:
> 	Add an option to control whether the compiler should try to use
> 	atomic cells.
>

...

That looks okay.

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