[PATCH] D23279: clang-reorder-fields

[Ben Craig via cfe-commits]

bcraig added a comment.

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

> @djasper, @bcraig, @aaron.ballman - many thanks for the comments and suggestions,
>  please, let me know if you are waiting for any other changes on my side (for v0) or if there is anything else i can do to make reviewing easier.


I'm not waiting on anything for a v0.  Again, just looking at the tests here.

I would like to see in a future revision (or this one) a test for the multi-TU support.  I would like the test to have one header with a struct definition, and two .c / .cpp files that use the struct definition in a way that would need to be modified.


Repository:
  rL LLVM

https://reviews.llvm.org/D23279