[m-dev.] I want to move and rename the dependency_graph module

Paul Bone paul at bone.id.au
Tue Feb 7 11:59:03 AEDT 2017


On Mon, Feb 06, 2017 at 10:53:44PM +1100, Julien Fischer wrote:
> 
> Hi Paul,
> 
> On Mon, 6 Feb 2017, Paul Bone wrote:
> 
> >compiler/dependency_graph.m has types and routines used to to build and use
> >a dependency graph of the HLDS.  I want to add the same for the MLDS.  To
> >avoid confusion I want to rename this module hlds.dependency_graph.m so I
> >also have mlds.dependency_graph.m
> 
> In keeping with the names of existing modules that define the HLDS and
> MLDS respectively I would name them hlds_dependency_graph.m and
> ml_dependency_graph.m.

Okay.

> >I also plan to extract the common code and probably put it in either
> >dependency_graph.m or libs.dependency_graph.m
> 
> What code will end up in common?  Would any of the common code be better
> placed in library/digraph.m?

Some of the types.

Perhaps some of it can go in the library.  I'll check this as I go.

> >The current dependency_graph module is part of the transform_hlds module.  I
> >would also like to move it to the hlds module.  I think it makes more sense
> >there since it's utility code rather than a transformation.
> 
> There is some sense in that as it's imported by other packages in the
> compiler, notably top_level and check_hlds.

That's what I thought.  There are a few cases were've I've been able to
remove the dependency on the entire transform_hlds module.

Cheers.


-- 
Paul Bone
http://paul.bone.id.au


More information about the developers mailing list