================
@@ -188,13 +192,23 @@ class FullDependencyConsumer : public DependencyConsumer {
     ContextHash = std::move(Hash);
   }
 
+  void handleProvidedAndRequiredStdCXXModules(
+      std::optional<P1689ModuleInfo> Provided,
+      std::vector<P1689ModuleInfo> Requires) override {
+    ModuleName = Provided ? Provided->ModuleName : "";
----------------
shafik wrote:

Static analysis flagged code around here and I am stuck trying to understand 
what `ModuleName` is being used for here. Is this dead code? If not there 
should really be a comment explaining how it is used.

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

Reply via email to