[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