================
@@ -115,6 +115,15 @@ struct ModuleID {
   }
 };
 
+struct ExtendedModuleID {
----------------
qiongsiwu wrote:

Hmmm I think I am misunderstanding something. Is this what we are proposing? 
Instead of 
```
  std::vector<ModuleDeps::DepInfo> ClangModuleDeps;
```

we have

```
  std::vector<std::pair<ModuleID, bool>> ClangModuleDeps;
```

It feels like using pairs is less expressive than naming the two fields in 
`DepInfo`. 

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