================ @@ -183,7 +192,7 @@ struct ModuleDeps { /// /// This may include modules with a different context hash when it can be /// determined that the differences are benign for this compilation. - std::vector<ModuleID> ClangModuleDeps; + std::vector<ExtendedModuleID> ClangModuleDeps; ---------------- qiongsiwu wrote:
I realized I might have misread this question. > Why is whether a module is Exported encoded here instead of directly as an > attribute in `ModuleDeps`? Are we discussing the possibility where we have a flag in `ModuleDeps ` that indicates if this current module is exported by its dependents..? https://github.com/llvm/llvm-project/pull/137421 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits