================
@@ -521,8 +529,14 @@ class FullDeps {
               } else {
                 JOS.object([&] {
                   JOS.attribute("clang-context-hash", 
StringRef(I.ContextHash));
+                  if (I.ModuleName.size())
+                    JOS.attribute("module-name", (I.ModuleName));
----------------
jansvoboda11 wrote:

It seems like the rest of the code tries to distinguish Clang and standard C++ 
modules. Maybe this name makes the distinction more obvious?
```suggestion
                    JOS.attribute("named-module", I.ModuleName);
```

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