[PATCH] D23279: clang-reorder-fields

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 9 01:48:56 PDT 2016


omtcyfz added a comment.

And actually it makes much more sense for C than for C++. In C++ you just do `s/struct/class/g`, insert `public:` and you're golden.

P.S. It actually breaks code. I haven't looked into that, but

  bar::Foo foo = { &x, 17, 1.29, 0 }; // CHECK: bar::Foo foo = { 1.29, 0, 17, &x };

only works while it is in the same TU.

+ @alexfh on `clang-tidy` discussion.


Repository:
  rL LLVM

https://reviews.llvm.org/D23279





More information about the cfe-commits mailing list