[m-rev.] for review: minimal model tabling analysis

Julien Fischer juliensf at cs.mu.OZ.AU
Fri Jun 2 13:06:40 AEST 2006


On Thu, 1 Jun 2006, Zoltan Somogyi wrote:

> > +:- pred make_opt_int(module_info::in, io::di, io::uo) is det.
> > +
> > +make_opt_int(ModuleInfo, !IO) :-
> > +    module_info_get_name(ModuleInfo, ModuleName),
> > +    module_name_to_file_name(ModuleName, ".opt.tmp", no, OptFileName, !IO),
> > +    globals.io_lookup_bool_option(verbose, Verbose, !IO),
> > +    maybe_write_string(Verbose, "% Appending mm_tabling_info pragmas to `",
> > +        !IO),
>
> Why is this a separate pass?

What do you mean, why is it separate from the rest of the tabling analysis
or why is separate from the intermodule-optimization code in intermod.m?

> > + at item @samp{will_not_call_mm_tabled/may_call_mm_tabled}
> > +This attribute declares whether or not a foreign procedure makes calls
> > +back to Mercury procedures that are evaluated using minimal model tabling
> > +(see @ref{Tabled evaluation}).  Specifying that a foreign procedure
> > +will not call procedures evaluated using minimal model tabling may allow
> > +the compiler to generate more efficient code.  In compilation grades that
> > +do not support minimal model tabling this attribute is ignored.  These
> > +attributes may not be used with procedures that do not make calls back
> > +to Mercury, i.e. that have the @samp{will_not_call_mercury} attribute.
> > +The default, in case none is specified, is @samp{may_call_mm_tabled}.

I've changed the last sentence to:

	The default for foreign procedures that @samp{may_call_mercury},
	in case none is specified, is @samp{may_call_mm_tabled}.

I've followed your other suggestions elsewhere.

Thanks for that.

Julien.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list