naveen-seth wrote:

For additional context:
Using the DOT dependency graph remark introduced in #152770, this patch changes 
the dependency scan result for the new regression test:

### Before this patch:
<p align="center">
  <img 
src="https://github.com/user-attachments/assets/c2d5ad60-de15-47dc-b62e-be1704f3032c";
 alt="graph-before-fix"/>
</p>

Because the driver-generated -cc1 command lines for C++ named module inputs 
affect the canonical module build command, the Clang module _root_ ends up with 
a different context hash for each import of a named module input.

### After this patch:
<p align="center">
  <img 
src="https://github.com/user-attachments/assets/4135d3e8-5726-4788-8d47-26a6330d8eca";
 alt="graph-after-fix"/>
</p>

https://github.com/llvm/llvm-project/pull/161486
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to