[PATCH] D23279: clang-reorder-fields

Ben Craig via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 22 13:34:33 PDT 2016


bcraig added a comment.

In https://reviews.llvm.org/D23279#522504, @djasper wrote:

> In the meantime, I don't know whether we should check this in as a standalone-tool and then merge into clang-refactor once ready. Ben, what do you think?


Well, I don't know how much you should count my opinion, but my preference is to check it in as a standalone tool first, then merge it into clang-rector once it is available.  Maybe the build gets turned off by default if we aren't comfortable shipping an interim tool.  I'd rather this not linger as a very long-term review though.


Repository:
  rL LLVM

https://reviews.llvm.org/D23279





More information about the cfe-commits mailing list