================
@@ -237,6 +238,8 @@ GetAffectingModuleMaps(const Preprocessor &PP, Module 
*RootModule) {
     CollectIncludingMapsFromAncestors(CurrentModule);
     for (const Module *ImportedModule : CurrentModule->Imports)
       CollectIncludingMapsFromAncestors(ImportedModule);
+    for (const Module *UsedModule : CurrentModule->DirectUses)
----------------
sam-mccall wrote:

I don't really understand what this change does, and all the tests pass without 
it. Is it related to the rest of this patch?

(Looking at the other collections that happen here, it does seem to "fit". But 
I don't have a really clear idea about this piece).

https://github.com/llvm/llvm-project/pull/89441
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to