[m-dev.] for review: bug fix for existential_rtti.m
Zoltan Somogyi
zs at cs.mu.OZ.AU
Tue Feb 15 19:15:01 AEDT 2000
> > compiler/type_util.m:
> > Add a new predicate, type_util__real_vars, which is the same
> > as type_util__vars except it doesn't count type-info variables.
> > compiler/{continuation_info.m, hlds_pred.m, trace.m}:
> > Use type_util__real_vars rather than type_util__vars since the
> > latter was causing problems when calculating the live type-infos,
> > and causing the failure of the test case
> > tests/hard_coded/existential_rtti.m in debug grade.
>
> All your changes look fine.
>
> The only thing I'm concerned about is that there
> may be some additional places which should be changed
> to use type_util__real_vars rather than type_util__vars
> or term__vars_list (or term__contains_var).
>
> But at very least your change is an improvement on the
> status quo, so go ahead and commit it.
I strongly suspect that all finding and fixing all those additional places
will be only be at least half as much work as making typeinfo a builtin type,
but that this work will need to be redone when we do make typeinfo a builtin type.
Zoltan.
--------------------------------------------------------------------------
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