zturner added a comment.

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.


https://reviews.llvm.org/D23409



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

Reply via email to