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

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 15 06:38:49 PDT 2016


klimek added a comment.

In https://reviews.llvm.org/D21814#478879, @vmiklos wrote:

> As far as I see `tooling::CommonOptionsParser` (in its current form) does not handle cl::SubCommand instances. Should I fix that or would it be OK to switch to using `cl::ParseCommandLineOptions` directly in clang-rename?


Well, if you switch, you'll lose a lot of what CommonOptionsParser gives you... So I think it's useful to keep it working with CommonOptionsParser.


https://reviews.llvm.org/D21814





More information about the cfe-commits mailing list