schaerfo added a comment.

In D90535#2366731 <https://reviews.llvm.org/D90535#2366731>, @njames93 wrote:

> Just a general drive by comment, there have been quite a few patches recently 
> that add arguments that just get forwarded straight to clang-tidy, would it 
> not be a whole lot simpler if we could just automatically forward arguments 
> straight to clang-tidy.

That is indeed an idea worth considering, perhaps I can start working on it. It 
would, however, not be backwards compatible.

> Anyway aside from that, please upload diffs with the full context, see 
> https://llvm.org/docs/Phabricator.html#phabricator-request-review-web for 
> more info.

I have updated the revision to show the entire diff.


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

https://reviews.llvm.org/D90535

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

Reply via email to