Skip to content

Commit 0b6abe4

Browse files
committed
[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.
1 parent c54597b commit 0b6abe4

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

llvm/include/llvm/module.modulemap

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -382,6 +382,9 @@ module LLVM_Utils {
382382

383383
umbrella "Support"
384384
module * { export * }
385+
386+
// Exclude this; it should only be used on Windows.
387+
exclude header "Support/Windows/WindowsSupport.h"
385388

386389
// Exclude these; they are fundamentally non-modular.
387390
exclude header "Support/PluginLoader.h"

0 commit comments

Comments
 (0)