ChuanqiXu added a comment.

> My GCC patch names it -fdepfile-output= instead of --p1689-targeted-output, 
> gets --p1689-targeted-file-name from the regular command line (it looks like 
> a preprocessor command overall) and the --p1689-makeformat-output flag is 
> handled by the normal -MT and -MF flags. While I'm not thrilled with the 
> P1689 <https://reviews.llvm.org/P1689> naming in the flags (as it'll become 
> P2222 <https://reviews.llvm.org/P2222> once the TR it's heading for is 
> accepted, aliases should be simple to maintain. My GCC patch uses 
> -fdep-format=p1689r5 so a rename there is easier than a whole flag name. But 
> I suspect that clang-scan-deps is much more of a "tooling's tool" than 
> something people type out by hand, so it's probably not a big deal?

Agreed. I suggest to keep the current name in the process of reviewing if 
@jansvoboda11 don't feel unhappy.


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