[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