[m-dev.] for review: options to switch off RTTI features.

Tyson Dowd trd at cs.mu.OZ.AU
Wed May 26 12:58:09 AEST 1999


On 25-May-1999, Zoltan Somogyi <zs at cs.mu.OZ.AU> wrote:
> 
> > Index: compiler/stack_layout.m
> > ===================================================================
> > RCS file: /home/mercury1/repository/mercury/compiler/stack_layout.m,v
> > retrieving revision 1.29
> > diff -u -r1.29 stack_layout.m
> > --- stack_layout.m	1999/05/07 08:08:56	1.29
> > +++ stack_layout.m	1999/05/25 02:33:54
> > @@ -768,10 +768,16 @@
> >  		initial(ArgTypes, none), must_be_static, CNum1,
> >  		"stack_layout_locn_vector") },
> >  
> > -	{ list__map(SelectNames, AllArrayInfo, AllNames) },
> > -	stack_layout__get_next_cell_number(CNum2),
> > -	{ NameVector = create(0, AllNames, uniform(yes(string)),
> > -		must_be_static, CNum2, "stack_layout_name_vector") }.
> > +	stack_layout__get_agc_stack_layout(AgcStackLayout),
> > +	( { AgcStackLayout = no } ->
> > +		{ list__map(SelectNames, AllArrayInfo, AllNames) },
> > +		stack_layout__get_next_cell_number(CNum2),
> > +		{ NameVector = create(0, AllNames, uniform(yes(string)),
> > +			must_be_static, CNum2, "stack_layout_name_vector") }
> > +	;
> > +		{ NameVector = const(int_const(0)) }
> > +	).
> > +
> 
> Your change looks good, except for the condition in the if-then-else above.
> You want to generate variable names if debugging is turned on, regardless
> of whether we are using agc or not, so test for that, not for agc.

Good call.

I've changed it to call
	stack_layout__get_trace_stack_layout(TraceStackLayout),
and generate the names if TraceStackLayout = yes.

-- 
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