ben.boeckel added inline comments.
================ Comment at: clang/test/ClangScanDeps/P1689.cppm:9 +// +// Check the seperated dependency format. +// RUN: clang-scan-deps -format=p1689 --p1689-targeted-file-name=%t/M.cppm --p1689-targeted-output=%t/M.o \ ---------------- jansvoboda11 wrote: > What does "separated" mean in this context? Yeah, this isn't the right term. There are two things being done: - discovering dependencies for a future compile (P1689) - collecting deps for the scanning itself to know that "if included file X changes, I need to rescan" Both are required for correct builds. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137534/new/ https://reviews.llvm.org/D137534 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
