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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to