[PATCH] D21814: clang-rename: support multiple renames with one invocation

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 7 14:24:45 PDT 2016


klimek added a comment.

Kirill, I think it's important to note that Miklos is probably a user, otherwise I'd guess he wouldn't add those features.
Generally, we also internally have a (large scale) simplified rename tool that allows renaming multiple things in a code base at once; it is definitely a useful tool to have.
I believe that Benjamin expresses the opinion of multiple people in the community (usually including Chandler) who want fewer binaries.
Perhaps we can make this better by doing the following compromise:
Instead of just having tons of parameters where it's hard to say which parameter does what, have 2 different commands that you can call, each with their own set of flags:
clang-rename rename-at -offset ... -newname ...
clang-rename rename-all -oldname ... -newname ... -oldname ... -new-name ... (or similar)


http://reviews.llvm.org/D21814





More information about the cfe-commits mailing list