[PATCH] D23409: Make clang-tidy work with clang-cl

Zachary Turner via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 11 11:10:37 PDT 2016


zturner added a comment.

In https://reviews.llvm.org/D23409#512745, @zturner wrote:

> In https://reviews.llvm.org/D23409#512720, @aaron.ballman wrote:
>
> > Patch generally LGTM, though I wonder if there's a way we can add test coverage for this.
>
>
> I'd imagine we can just enable the clang-tidy test suite on Windows (I'm assuming it's currently disabled).  I'll look into it.


Actually yea I'm not sure how to go about doing this.  All the existing tests are written to use GCC style command line, and it seems unreasonable to add a cl style command line variant of every single test.  maybe alexfh@ or someone has a suggestion.


https://reviews.llvm.org/D23409





More information about the cfe-commits mailing list