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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
