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

[Benjamin Kramer via cfe-commits]

bkramer added a comment.

In http://reviews.llvm.org/D21814#475322, @klimek wrote:

> I think we really want 2 tools:
>  a) one that is optimized for oldname->newname renames, and supports the multi-TU case really well
>  b) one that is meant to be integrated with editors and works mainly off of a location in a file
>
> I'm a bit torn whether putting those 2 into the same executable is a good idea. Looping in Benjamin for additional ideas.


I'm fine with both things living in the same binary. The location thing is just a different way of specifying the symbol name. I fail to see how that's related to this review though.


http://reviews.llvm.org/D21814