[llvm] 0b6abe4 - [llvm][Support][modulemap] Exclude WindowsSupport.h from the LLVM_Util module

[Michael Spencer via llvm-commits]

Author: Michael Spencer
Date: 2020-02-28T14:40:14-08:00
New Revision: 0b6abe428164c38c7dec6c539e454cfd671067a4

URL: https://github.com/llvm/llvm-project/commit/0b6abe428164c38c7dec6c539e454cfd671067a4
DIFF: https://github.com/llvm/llvm-project/commit/0b6abe428164c38c7dec6c539e454cfd671067a4.diff

LOG: [llvm][Support][modulemap] Exclude WindowsSupport.h from the LLVM_Util module

rG01f9abbb50b1 moved WindowsSupport.h to include/llvm/Support/Windows/

This is a problem because the modulemap include all of the Support and
ADT directories, thus any use of any header in Support or ADT would
cause the compiler to try to build WindowsSupport.h, which only works
on Windows.

Fix this by explicitly excluding WindowsSupport.h from the LLVM_Util
module.

Added: 
    

Modified: 
    llvm/include/llvm/module.modulemap

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/module.modulemap b/llvm/include/llvm/module.modulemap
index d220cc0ab0be..f3623a6ff5da 100644
--- a/llvm/include/llvm/module.modulemap
+++ b/llvm/include/llvm/module.modulemap
@@ -382,6 +382,9 @@ module LLVM_Utils {
 
     umbrella "Support"
     module * { export * }
+    
+    // Exclude this; it should only be used on Windows.
+    exclude header "Support/Windows/WindowsSupport.h"
 
     // Exclude these; they are fundamentally non-modular.
     exclude header "Support/PluginLoader.h"