[PATCH] D137534: [C++20] [Modules] [ClangScanDeps] Allow clang-scan-deps to without specified compilation database in P1689 (3/4)

[Ben Boeckel via Phabricator via cfe-commits]

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