From 0b6abe428164c38c7dec6c539e454cfd671067a4 Mon Sep 17 00:00:00 2001 From: Michael Spencer Date: Fri, 28 Feb 2020 14:39:49 -0800 Subject: [PATCH] [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. --- llvm/include/llvm/module.modulemap | 3 +++ 1 file changed, 3 insertions(+) diff --git a/llvm/include/llvm/module.modulemap b/llvm/include/llvm/module.modulemap index d220cc0..f3623a6 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" -- 2.7.4