[PATCH] D21814: clang-rename: split existing options into two new subcommands

[Miklos Vajna via cfe-commits]

vmiklos added a comment.

In https://reviews.llvm.org/D21814#500621, @omtcyfz wrote:

> P.S. not sure whether we have to write `clang-rename: for the -new-name option: must be specified` out. We already launched `clang-rename` what else could've give us an error?


You mean how is that error produced? Previously there was no `cl::Required` on NewName (now: NewNames), and that's why a manual check was needed, the error message changed, as now the option parser does this checking for us.


https://reviews.llvm.org/D21814