================
@@ -71,6 +71,7 @@
 // CHECK-NEXT:       "context-hash": "{{.*}}",
 // CHECK-NEXT:       "file-deps": [
 // CHECK-NEXT:         "[[PREFIX]]/first/module.modulemap",
+// CHECK-NEXT:         "[[PREFIX]]/second/module.modulemap",
----------------
jansvoboda11 wrote:

This is a side-effect of the change in `CompilerInstance.cpp`. The implicit 
scan now uses the top-level module map to compile modules instead of the 
containing module map and that makes it included in file dependencies. This 
change is a bit unfortunate, since the explicit command line we generate 
actually only uses the containing module map, not the top-level one. Ideally, 
we'd either find a way to remove this file dependency or we'd make the explicit 
compile to also consume the top-level module map (in which case this file 
dependency would be real).

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

Reply via email to