Re: [PATCH] D21120: clang-rename: implement renaming of classes inside dynamic_cast

2016-06-09 Thread Miklos Vajna via cfe-commits
vmiklos added a comment. Possible solution posted as http://reviews.llvm.org/D21186. Repository: rL LLVM http://reviews.llvm.org/D21120 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D21120: clang-rename: implement renaming of classes inside dynamic_cast

2016-06-08 Thread Miklos Vajna via cfe-commits
vmiklos added a comment. Hmm, http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast/builds/14332/steps/test/logs/stdio fails with "error: cannot use dynamic_cast with -fno-rtti"; is there some way to detect if "-fno-rtti" is the default, and in that case don't execute

Re: [PATCH] D21120: clang-rename: implement renaming of classes inside dynamic_cast

2016-06-08 Thread Miklos Vajna via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL272188: clang-rename: implement renaming of classes inside dynamic_cast (authored by vmiklos). Changed prior to commit: http://reviews.llvm.org/D21120?vs=60001=60073#toc Repository: rL LLVM

Re: [PATCH] D21120: clang-rename: implement renaming of classes inside dynamic_cast

2016-06-08 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/D21120 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D21120: clang-rename: implement renaming of classes inside dynamic_cast

2016-06-08 Thread Miklos Vajna via cfe-commits
vmiklos created this revision. vmiklos added a reviewer: klimek. vmiklos added a subscriber: cfe-commits. Refactor to do the same as what is done already for static_cast. http://reviews.llvm.org/D21120 Files: clang-rename/USRLocFinder.cpp test/clang-rename/DynamicCastExpr.cpp Index: