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

2016-06-21 Thread Miklos Vajna via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL273304: clang-rename: add a -old-name option (authored by vmiklos). Changed prior to commit: http://reviews.llvm.org/D21517?vs=61255&id=61430#toc Repository: rL LLVM http://reviews.llvm.org/D21517

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

2016-06-21 Thread Manuel Klimek via cfe-commits
klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. LG http://reviews.llvm.org/D21517 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm

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

2016-06-20 Thread Miklos Vajna via cfe-commits
vmiklos updated this revision to Diff 61255. vmiklos added a comment. Added a getNamedDeclFor() for the fully qualified name-based search. http://reviews.llvm.org/D21517 Files: clang-rename/USRFinder.cpp clang-rename/USRFinder.h clang-rename/USRFindingAction.cpp clang-rename/USRFindingA

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

2016-06-20 Thread Manuel Klimek via cfe-commits
klimek added inline comments. Comment at: clang-rename/USRFinder.h:28-33 @@ -27,6 +27,8 @@ -// Given an AST context and a point, returns a NamedDecl identifying the symbol -// at the point. Returns null if nothing is found at the point. +// Given an AST context and a point (or f