Skip to content

Commit

Permalink
[llvm][Support][modulemap] Exclude WindowsSupport.h from the LLVM_Uti…
Browse files Browse the repository at this point in the history
…l 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.

(cherry picked from commit 0b6abe4)
  • Loading branch information
Bigcheese authored and zmodem committed Mar 2, 2020
1 parent dce2ef9 commit 5405c26
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions llvm/include/llvm/module.modulemap
Original file line number Diff line number Diff line change
Expand Up @@ -380,6 +380,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"
Expand Down

0 comments on commit 5405c26

Please sign in to comment.