================
@@ -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

Reply via email to