[clang] [clang][modules] Allow including module maps to be non-affecting (PR #89992)

[Ben Langmuir via cfe-commits]

================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits