[PATCH] D23279: clang-reorder-fields

Ben Craig via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 10 07:57:14 PDT 2016

bcraig added a comment.

In https://reviews.llvm.org/D23279#511187, @omtcyfz wrote:

> In https://reviews.llvm.org/D23279#510234, @alexshap wrote:
> > > I do think that an automated tool will do a better job of changing field orderings in a non-breaking way than most people would, mostly due to initializer lists.
> >
> >
> > yeah, that was the original motivation
> How does that justify its current state, in which it does breaks codebases with more than 1 TU?

Yeah, that aspect probably needs work :).  Accepting a compilation database seems like a good way to correct that.



More information about the cfe-commits mailing list