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

[Chuanqi Xu via Phabricator via cfe-commits]

ChuanqiXu added a comment.

In D137534#3965012 <https://reviews.llvm.org/D137534#3965012>, @ben.boeckel wrote:

> So a few things I see when trying to put this into CMake:
>
> - P1689 <https://reviews.llvm.org/P1689> output is only to `stdout`?
> - Is there a way to get `-MF` output for the files read during scanning? This is useful to know that "header X changed, scanning may have changed, so please rerun scanning".
>
> Other than that, I have success with CMake's test suite with this patchset as well as the patchset of D137059 <https://reviews.llvm.org/D137059>.

Great to hear that!

> - P1689 <https://reviews.llvm.org/P1689> output is only to `stdout`?

>From the current implementation, it looks true that the output of clang-scan-deps is only to stdout. I guess it may not be problem since we can redirect it to the file we want. Or is it necessary to specify an output file in the flags?

> Is there a way to get `-MF` output for the files read during scanning? This is useful to know that "header X changed, scanning may have changed, so please rerun scanning".

I sent https://reviews.llvm.org/D139168 to address. You can specify the output of `-MF` in `--p1689-makeformat-output=` flags. Is this satisfying?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137534/new/

https://reviews.llvm.org/D137534