[PATCH] D23279: clang-reorder-fields

Saleem Abdulrasool via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 8 14:15:52 PDT 2016


compnerd added a comment.

This isn't really a renaming tool per se.  If you squint really hard, yes, it does rename fields.  But, if we really want to save space, perhaps we should collapse all the tools into `clang-tidy` or create a new `clang-refactor` tool and just make the other things a part of that tool in various modes (rename, reorder-fields, extract, etc) via sub-commands (a la git).  However, I think thats a broader design decision which could be made outside the context of this change.  However, if the concern is purely for install-time, we could add components to the CMake install to control which of the extra tools are built (note that this change doesn't even install the new binary!).


Repository:
  rL LLVM

https://reviews.llvm.org/D23279





More information about the cfe-commits mailing list