ben.boeckel added inline comments.
================ Comment at: clang/test/ClangScanDeps/P1689.cppm:155 +// CHECK-MAKE: [[PREFIX]]/impl_part.o: +// CHECK-MAKE: [[PREFIX]]/impl_part.cppm ---------------- For scanning, this cannot be the object file. The output needs to be the P1689 output (or whatever the "main output" for the scanning rule is). This is the purpose behind the `-MT <target>` flag: to say what goes in this slot. I think it'll be necessary here (even if `clang-scan-deps` learns an `-o` flag because it is the build system that determines the "primary output"). I don't know if the `-MMD` and `-MD` differences are important or not; I don't think I particularly care which is default (I've used `-MD` FWIW), but it may matter for other situations. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139168/new/ https://reviews.llvm.org/D139168 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
