[PATCH] D23279: clang-reorder-fields

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 22 18:40:26 PDT 2016


aaron.ballman added a comment.

In https://reviews.llvm.org/D23279#522779, @alexshap wrote:

> @aaron.ballman
>  Thanks, yeah, there is an issue. Also there are inline-comments by @bcraig and @compnerd about the tests.
>  I used clang-rename, include-fixer, clang-tidy as examples.


Correct, but those tools have other tests that ensure they work on Windows (though those features are unfortunately untested), while this tool has *zero* tests that will run on Windows.


Repository:
  rL LLVM

https://reviews.llvm.org/D23279





More information about the cfe-commits mailing list