r230600 - Split StaticAnalyzer module into three to fix a cyclic dependency. Dependencies

Richard Smith richard-llvm at metafoo.co.uk
Wed Feb 25 17:11:08 PST 2015


Author: rsmith
Date: Wed Feb 25 19:11:08 2015
New Revision: 230600

URL: http://llvm.org/viewvc/llvm-project?rev=230600&view=rev
Log:
Split StaticAnalyzer module into three to fix a cyclic dependency. Dependencies
are now:

  FrontendTool -> StaticAnalyzer/Frontend -> Frontend -> StaticAnalyzer/Core

The final dependency edge here is probably removable: AnalyzerOptions (and
Analyses.def) should probably live in Basic rather than StaticAnalyzer/Core.

Modified:
    cfe/trunk/include/clang/module.modulemap

Modified: cfe/trunk/include/clang/module.modulemap
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/module.modulemap?rev=230600&r1=230599&r2=230600&view=diff
==============================================================================
--- cfe/trunk/include/clang/module.modulemap (original)
+++ cfe/trunk/include/clang/module.modulemap Wed Feb 25 19:11:08 2015
@@ -98,13 +98,25 @@ module Clang_Rewrite { requires cplusplu
 module Clang_Sema { requires cplusplus umbrella "Sema" module * { export * } }
 module Clang_Serialization { requires cplusplus umbrella "Serialization" module * { export * } }
 
-module Clang_StaticAnalyzer {
+module Clang_StaticAnalyzer_Core {
   requires cplusplus
-  umbrella "StaticAnalyzer"
+  umbrella "StaticAnalyzer/Core"
 
   textual header "StaticAnalyzer/Core/Analyses.def"
 
   module * { export * }
 }
 
+module Clang_StaticAnalyzer_Checkers {
+  requires cplusplus
+  umbrella "StaticAnalyzer/Checkers"
+  module * { export * }
+}
+
+module Clang_StaticAnalyzer_Frontend {
+  requires cplusplus
+  umbrella "StaticAnalyzer/Frontend"
+  module * { export * }
+}
+
 module Clang_Tooling { requires cplusplus umbrella "Tooling" module * { export * } }





More information about the cfe-commits mailing list