[m-rev.] for review: dead_pred_elim optimization
Fergus Henderson
fjh at cs.mu.OZ.AU
Wed Apr 16 11:00:58 AEST 2003
On 15-Apr-2003, Peter Ross <pro at missioncriticalit.com> wrote:
> Fix a bug in the implementation of predicate_table_restrict where we
> were assuming that because a procedure was not accessible by
> unqualified name that it was also not accessible by partially
> qualified names.
>
> compiler/hlds_pred.m:
> Replace the marker only_accessible_via_fully_qualifed_name
> with the markers not_accessible_by_unqualifed_name and
> not_accessible_by_partially_qualified_names.
Isn't this just going to fail in cases involving nested modules
which are nested one more level?
E.g. consider a name "a.b.c.d.e.f", where "a", "b", and "d"
have been imported using "import_module", and "c" and "e" have
been imported using "use_module". Valid ways to refer to this
name are "c.d.e.f", "b.c.d.e.f", and "a.b.c.d.e.f",
but not "d.e.f", "e.f", or "f".
Will your code handle this case correctly?
--
Fergus Henderson <fjh at cs.mu.oz.au> | "I have always known that the pursuit
The University of Melbourne | of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh> | -- the last words of T. S. Garp.
--------------------------------------------------------------------------
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