[PATCH] D23279: clang-reorder-fields

[Alexander Shaposhnikov via cfe-commits]

alexshap removed rL LLVM as the repository for this revision.
alexshap updated this revision to Diff 68738.
alexshap added a comment.

1. Handle C++ ctors.
2. Add tests.
3. I have a diff for proper moving the attached comments

based on http://llvm.org/devmtg/2012-11/Gribenko_CommentParsing.pdf .
It is not ready yet but it's in the oven.

Yes, of course i remember about the planned higher-level reorganization regarding clang-refactor,
although it would be helpful i could make some progress incrementally and then merge it into clang-refactor (or if you can create the corresponding folders tree i could try to put this stuff there and disable its build in cmake files). The main purpose - i think it might be helpful to evolve this stuff iteratively & smaller diffs instead of one gigantic.  P.S. I have used this tool to remove excessive padding from several structs/classes in the LLVM code where it was appropriate.


https://reviews.llvm.org/D23279

Files:
  CMakeLists.txt
  clang-reorder-fields/CMakeLists.txt
  clang-reorder-fields/ReorderFieldsAction.cpp
  clang-reorder-fields/ReorderFieldsAction.h
  clang-reorder-fields/tool/CMakeLists.txt
  clang-reorder-fields/tool/ClangReorderFields.cpp
  test/CMakeLists.txt
  test/clang-reorder-fields/CStructAmbiguousName.cpp
  test/clang-reorder-fields/CStructFieldPos.cpp
  test/clang-reorder-fields/CStructFieldsOrder.cpp
  test/clang-reorder-fields/ClassMixedInitialization.cpp
  test/clang-reorder-fields/ClassSimpleCtor.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23279.68738.patch
Type: text/x-patch
Size: 25775 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160819/beda971e/attachment-0001.bin>