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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to