omtcyfz added a comment. In https://reviews.llvm.org/D21814#500629, @vmiklos wrote:
> 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. Ah, I see. So the message is generated, isn't it? https://reviews.llvm.org/D21814 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
