[clang] [clang][modules] Allow including module maps to be non-affecting (PR #89992)
Ben Langmuir via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 26 14:18:24 PDT 2024
================
@@ -249,9 +245,27 @@ GetAffectingModuleMaps(const Preprocessor &PP, Module *RootModule) {
for (const auto &KH : HS.findResolvedModulesForHeader(*File))
if (const Module *M = KH.getModule())
- CollectIncludingMapsFromAncestors(M);
+ CollectModuleMapsForHierarchy(M);
}
+ // FIXME: This algorithm is not correct for module map hierarchies where
----------------
benlangmuir wrote:
Was this working before, or always broken?
https://github.com/llvm/llvm-project/pull/89992
More information about the cfe-commits
mailing list