================ @@ -57,6 +57,10 @@ struct TranslationUnitDeps { /// determined that the differences are benign for this compilation. std::vector<ModuleID> ClangModuleDeps; + /// A list of module names that are visible to this translation unit. This + /// includes both direct and transitive module dependencies. + std::vector<std::string> VisibleModules; ---------------- cyndyishida wrote:
Thank you for checking this😅 hm, so I definitely want to avoid querying for the visible modules for each transitive module dep, because what only matters is the "main" as you said, so I'm inclined to avoid adding the field to the ModuleDep object itself because it would be confusing when it's populated & when it's not. I think the easiest solution would be to change the return type, since the only significant difference AFAICT between those APIs is `getModuleDependencies` just creates an implicit buffer to be able to treat the request like a TU. https://github.com/llvm/llvm-project/pull/147969 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits