[m-rev.] for post-commit review: clarify some used_modules code
Peter Wang
novalazy at gmail.com
Wed Mar 30 11:09:07 AEDT 2022
On Wed, 30 Mar 2022 02:18:44 +1100 "Zoltan Somogyi" <zoltan.somogyi at runbox.com> wrote:
> Clarity some code for computing used modules.
>
> compiler/hlds_module.m:
> Document the semantics of the used_module field, and describe a possible
> bug.
>
> Give a predicate for recording modules as used a more descriptive name.
>
> compiler/make_hlds_passes.m:
> Conform to the change in hlds_module, and factor out both
> common code and common comments.
>
> compiler/prog_data_used_modules.m:
> Move an invariant test on visibility out of a loop.
That's fine.
Peter
More information about the reviews
mailing list