[llvm] 8d15e22 - Fix modules build after introduction of PassSupport.h include check

Raphael Isemann via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 04:54:04 PDT 2020


Author: Raphael Isemann
Date: 2020-04-27T13:53:44+02:00
New Revision: 8d15e222da24e5d3b284d0df4c72b3f0bdf76a9b

URL: https://github.com/llvm/llvm-project/commit/8d15e222da24e5d3b284d0df4c72b3f0bdf76a9b
DIFF: https://github.com/llvm/llvm-project/commit/8d15e222da24e5d3b284d0df4c72b3f0bdf76a9b.diff

LOG: Fix modules build after introduction of PassSupport.h include check

D78815 added a check that ensures that PassSupport.h and PassAnalysisSupport.h
aren't included directly. However, as compiling this header into a module will
try to parse it with a clean preprocessor state, this check is also triggered
there. The check seems to make sense and it seems that compiling this header
into its own module is a mistake, so this patch makes those two headers
textual in the modulemap.

Added: 
    

Modified: 
    llvm/include/llvm/module.modulemap

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/module.modulemap b/llvm/include/llvm/module.modulemap
index 223e705f0045..baab05c0928c 100644
--- a/llvm/include/llvm/module.modulemap
+++ b/llvm/include/llvm/module.modulemap
@@ -212,8 +212,8 @@ module LLVM_Pass {
     // PassSupport.h and PassAnalysisSupport.h are made available only through
     // Pass.h.
     header "Pass.h"
-    header "PassSupport.h"
-    header "PassAnalysisSupport.h"
+    textual header "PassSupport.h"
+    textual header "PassAnalysisSupport.h"
     export *
   }
 


        


More information about the llvm-commits mailing list