[llvm] r272877 - [modules] Combine Pass.h, PassSupport.h and PassAnalysisSupport.h into one module.

Vassil Vassilev via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 16 01:00:32 PDT 2016


Author: vvassilev
Date: Thu Jun 16 03:00:29 2016
New Revision: 272877

URL: http://llvm.org/viewvc/llvm-project?rev=272877&view=rev
Log:
[modules] Combine Pass.h, PassSupport.h and PassAnalysisSupport.h into one module.

The header files are designed to be used always together (through Pass.h).

Addresses the first part of https://llvm.org/bugs/show_bug.cgi?id=27991

Patch by Cristina Cristescu and me.

Reviewed by Richard Smith.

Modified:
    llvm/trunk/include/llvm/module.modulemap

Modified: llvm/trunk/include/llvm/module.modulemap
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/module.modulemap?rev=272877&r1=272876&r2=272877&view=diff
==============================================================================
--- llvm/trunk/include/llvm/module.modulemap (original)
+++ llvm/trunk/include/llvm/module.modulemap Thu Jun 16 03:00:29 2016
@@ -101,15 +101,22 @@ module LLVM_ExecutionEngine {
   exclude header "ExecutionEngine/OrcMCJITReplacement.h"
 }
 
-module LLVM_IR {
-  requires cplusplus
+module LLVM_Pass {
+  module Pass {
+    // PassSupport.h and PassAnalysisSupport.h are made available only through
+    // Pass.h.
+    header "Pass.h"
+    header "PassSupport.h"
+    header "PassAnalysisSupport.h"
+    export *
+  }
 
-  // FIXME: Is this the right place for these?
-  module Pass { header "Pass.h" export * }
-  module PassSupport { header "PassSupport.h" export * }
-  module PassAnalysisSupport { header "PassAnalysisSupport.h" export * }
   module PassRegistry { header "PassRegistry.h" export * }
   module InitializePasses { header "InitializePasses.h" export * }
+}
+
+module LLVM_IR {
+  requires cplusplus
 
   umbrella "IR"
   module * { export * }




More information about the llvm-commits mailing list