[m-dev.] for review: --shape-includes-inst

Tyson Dowd trd at cs.mu.OZ.AU
Tue Apr 20 15:47:46 AEST 1999


On 19-Apr-1999, Zoltan Somogyi <zs at cs.mu.OZ.AU> wrote:
> 
> For review by Tyson.
> 
> Estimated hours taken: 1
> 
> Reduce the size of executables compiled with tracing by not including
> space for instantiation information in label layout structures by default,
> but only when asked for. Previously, we allocated the space in all cases,
> even though there is no code yet for filling it in with real info.
> 
> This saves 0.1% for the compiler on hydra. That's not much, but it's something.
> 
> compiler/options.m:
> 	Add a new option, --shape-includes-inst, that causes the compiler
> 	to reserve space for insts in shapes. Only Tyson and whoever else
> 	works on adding inst info to the RTTI need ever use this option.
> 
> compiler/stack_layout.m:
> 	Only reserve space for insts if this option is set.
> 
> compiler/mercury_compile.m:
> 	If this option is set, pass -DMR_SHAPE_INCLUDES_INST to the C
> 	compiler.
> 
> runtime/mercury_stack_layout.h:
> 	Only include inst info in MR_Var_Shape_Info structs if
> 	MR_SHAPE_INCLUDES_INST is defined.

Apart from the conflicts, the only problem I can see with this is
that you haven't added MR_SHAPE_INCLUDES_INST to mercury_conf_param.h

-- 
The quantum sort: 
	while (!sorted) { do_nothing(); }
Tyson Dowd   <tyson at tyse.net>   http://tyse.net/
--------------------------------------------------------------------------
mercury-developers mailing list
Post messages to:       mercury-developers at cs.mu.oz.au
Administrative Queries: owner-mercury-developers at cs.mu.oz.au
Subscriptions:          mercury-developers-request at cs.mu.oz.au
--------------------------------------------------------------------------



More information about the developers mailing list