llvm-build: Validate information on the loaded components and form the topological...

 - Currently we require that all references between components (except the parent relation) fit into a DAG -- this could be relaxed later if it ever proves to be useful.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@143623 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/utils/llvm-build/llvmbuild/componentinfo.py b/utils/llvm-build/llvmbuild/componentinfo.py
index e15dbda756f9e3796cb9f497534b2bc2919026b2..22ce0b4fde72569d2a2321673c2e598bae62b149 100644 (file)
--- a/utils/llvm-build/llvmbuild/componentinfo.py
+++ b/utils/llvm-build/llvmbuild/componentinfo.py
@@ -37,6 +37,17 @@ class ComponentInfo(object):
         # under.
         self.parent = parent
 
         # under.
         self.parent = parent
 

+    def get_component_references(self):
+        """get_component_references() -> iter
+
+        Return an iterator over the named references to other components from
+        this object. Items are of the form (reference-type, component-name).
+        """
+
+        # Parent references are handled specially.
+        for r in self.dependencies:
+            yield ('dependency', r)
+

 class GroupComponentInfo(ComponentInfo):
     """
     Group components have no semantics as far as the build system are concerned,
 class GroupComponentInfo(ComponentInfo):
     """
     Group components have no semantics as far as the build system are concerned,


@@ -81,6 +92,14 @@ class LibraryComponentInfo(ComponentInfo):
         # considered part of.
         self.add_to_library_groups = list(add_to_library_groups)
 
         # considered part of.
         self.add_to_library_groups = list(add_to_library_groups)
 

+    def get_component_references(self):
+        for r in ComponentInfo.get_component_references(self):
+            yield r
+        for r in self.required_libraries:
+            yield ('required library', r)
+        for r in self.add_to_library_groups:
+            yield ('library group', r)
+

 class LibraryGroupComponentInfo(ComponentInfo):
     type_name = 'LibraryGroup'
 
 class LibraryGroupComponentInfo(ComponentInfo):
     type_name = 'LibraryGroup'
 


@@ -104,6 +123,14 @@ class LibraryGroupComponentInfo(ComponentInfo):
         # considered part of.
         self.add_to_library_groups = list(add_to_library_groups)
 
         # considered part of.
         self.add_to_library_groups = list(add_to_library_groups)
 

+    def get_component_references(self):
+        for r in ComponentInfo.get_component_references(self):
+            yield r
+        for r in self.required_libraries:
+            yield ('required library', r)
+        for r in self.add_to_library_groups:
+            yield ('library group', r)
+

 class ToolComponentInfo(ComponentInfo):
     type_name = 'Tool'
 
 class ToolComponentInfo(ComponentInfo):
     type_name = 'Tool'
 


@@ -121,6 +148,12 @@ class ToolComponentInfo(ComponentInfo):
         # tool.
         self.required_libraries = list(required_libraries)
 
         # tool.
         self.required_libraries = list(required_libraries)
 

+    def get_component_references(self):
+        for r in ComponentInfo.get_component_references(self):
+            yield r
+        for r in self.required_libraries:
+            yield ('required library', r)
+

 class BuildToolComponentInfo(ToolComponentInfo):
     type_name = 'BuildTool'
 
 class BuildToolComponentInfo(ToolComponentInfo):
     type_name = 'BuildTool'
 

diff --git a/utils/llvm-build/llvmbuild/main.py b/utils/llvm-build/llvmbuild/main.py
index 57233a070e3bff7e0b9022cf92955d2c1d5f6e56..a734ac16414c418d9da2519ab5a78998ae551b62 100644 (file)
--- a/utils/llvm-build/llvmbuild/main.py
+++ b/utils/llvm-build/llvmbuild/main.py
@@ -1,8 +1,12 @@
-import pprint

 import os
 import os

+import sys

 
 import componentinfo
 
 
 import componentinfo
 

+from util import *
+
+###
+

 class LLVMProjectInfo(object):
     @staticmethod
     def load_infos_from_path(llvmbuild_source_root):
 class LLVMProjectInfo(object):
     @staticmethod
     def load_infos_from_path(llvmbuild_source_root):


@@ -35,9 +39,62 @@ class LLVMProjectInfo(object):
         return LLVMProjectInfo(source_root, infos)
 
     def __init__(self, source_root, component_infos):
         return LLVMProjectInfo(source_root, infos)
 
     def __init__(self, source_root, component_infos):

+        # Store our simple ivars.

         self.source_root = source_root
         self.component_infos = component_infos
 
         self.source_root = source_root
         self.component_infos = component_infos
 

+        # Create the component info map and validate that component names are
+        # unique.
+        self.component_info_map = {}
+        for ci in component_infos:
+            existing = self.component_info_map.get(ci.name)
+            if existing is not None:
+                # We found a duplicate component name, report it and error out.
+                fatal("found duplicate component %r (at %r and %r)" % (
+                        ci.name, ci.subpath, existing.subpath))
+            self.component_info_map[ci.name] = ci
+
+        # Topologically order the component information according to their
+        # component references.
+        def visit_component_info(ci, current_stack, current_set):
+            # Check for a cycles.
+            if ci in current_set:
+                # We found a cycle, report it and error out.
+                cycle_description = ' -> '.join(
+                    '%r (%s)' % (ci.name, relation)
+                    for relation,ci in current_stack)
+                fatal("found cycle to %r after following: %s -> %s" % (
+                        ci.name, cycle_description, ci.name))
+
+            # If we have already visited this item, we are done.
+            if ci not in components_to_visit:
+                return
+
+            # Otherwise, mark the component info as visited and traverse.
+            components_to_visit.remove(ci)
+
+            for relation,referent_name in ci.get_component_references():
+                # Validate that the reference is ok.
+                referent = self.component_info_map.get(referent_name)
+                if referent is None:
+                    fatal("component %r has invalid reference %r (via %r)" % (
+                            ci.name, referent_name, relation))
+
+                # Visit the reference.
+                current_stack.append((relation,ci))
+                current_set.add(ci)
+                visit_component_info(referent, current_stack, current_set)
+                current_set.remove(ci)
+                current_stack.pop()
+
+            # Finally, add the component info to the ordered list.
+            self.ordered_component_infos.append(ci)
+
+        self.ordered_component_infos = []
+        components_to_visit = set(component_infos)
+        while components_to_visit:
+            visit_component_info(iter(components_to_visit).next(), [], set())
+

 def main():
     from optparse import OptionParser, OptionGroup
     parser = OptionParser("usage: %prog [options]")
 def main():
     from optparse import OptionParser, OptionGroup
     parser = OptionParser("usage: %prog [options]")