[PATCH] D21814: clang-rename: split existing options into two new subcommands
Miklos Vajna via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 15 15:52:14 PDT 2016
vmiklos added a comment.
I've implemented the requested split of options, now there are two new rename-at and rename-all subcommands. The only common options is -i and -new-name, nothing else is shared, apart from the common options added by `tooling::CommonOptionsParser`. The code is modeled after `llvm-cov`, that way I could break the chicken-and-egg problem I outlined in my previous comment.
I've also updated all tests to use either of the two subcommands.
For now I did not touch documentation, though that'll be obviously the next step, I just didn't want to make this patch larger than necessary. :-)
https://reviews.llvm.org/D21814
More information about the cfe-commits
mailing list