[PATCH] D21517: clang-rename: add a -old-name option

Miklos Vajna via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 20 01:31:06 PDT 2016


vmiklos created this revision.
vmiklos added a reviewer: klimek.
vmiklos added a subscriber: cfe-commits.

This is similar to -offset with the following differences:
    
1) -offset can refer to local variables as well.
    
2) -old-name makes it easier to refer to e.g. ClassName::MemberName by
spelling out the fully qualified name, instead of having to use e.g.
grep to look up the exact offset.
    
In other words, -offset is great when clang-rename is invoked by e.g. an
IDE, but not really user-friendly when the tool is invoked by the user
from commandline.  That's the use case where -old-name is supposed to
improve the situation.

http://reviews.llvm.org/D21517

Files:
  clang-rename/USRFinder.cpp
  clang-rename/USRFinder.h
  clang-rename/USRFindingAction.cpp
  clang-rename/USRFindingAction.h
  clang-rename/tool/ClangRename.cpp
  test/clang-rename/ClassTestByName.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21517.61237.patch
Type: text/x-patch
Size: 6083 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160620/c6dfce2d/attachment.bin>


More information about the cfe-commits mailing list