[LLVMbugs] [Bug 794] NEW: Factor Dominators.cpp, Verifier.cpp and Pass* out of lib/LLVMCore

bugzilla-daemon at cs.uiuc.edu bugzilla-daemon at cs.uiuc.edu
Wed May 31 11:49:58 PDT 2006


           Summary: Factor Dominators.cpp, Verifier.cpp and Pass* out of
           Product: libraries
           Version: 1.0
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Core LLVM classes
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: rspencer at x10sys.com

Not every user of lib/VMCore wants to run the verifier or use Passes. While
these facilities are common, they are not intrinsically part of the LLVM IR. In
particular, nothing in rest of lib/LLVMCore uses the Dominators classes other
than the Verifier using ET-Forest. The Pass.cpp and PassManagerT.h are
standalone and its only users in lib/VMCore are Verifier.cpp and AsmWriter.cpp,
both of which are targeted for factoring out of lib/VMCore.

It is suggested that the following changes be made:

1. Move Pass.cpp and PassManagerT.h to a new library, lib/Pass
2. Split ET-Forest stuff out of Dominators.h and Dominators.cpp and into its own
   module ETForest.h and ETForest.cpp. Not sure where this should live but it is
   used by the Verifier pass so could go where Verifier.cpp lives or perhaps
   in lib/Analysis.
3. Move Dominators.cpp to lib/Analysis
4. Move Verifier.cpp to lib/Analysis

This is the third in a set of enhancements to reduce the size of lib/VMCore.

Thoughts, ideas?


------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

More information about the llvm-bugs mailing list