[PATCH] D74829: [clang-rename] Add the USR of incomplete decl to the USRSet.

2020-02-25 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGe6d0bad843c4: [clang-rename] Add the USR of incomplete decl to the USRSet. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74829/new/ ht

[PATCH] D74829: [clang-rename] Add the USR of incomplete decl to the USRSet.

2020-02-21 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev added a comment. Yeah, it works in Clang-Rename due to additional steps in the renaming process. Testing only Clangd's rename is sufficient here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74829/new/ https://reviews.llvm.org/D74829

[PATCH] D74829: [clang-rename] Add the USR of incomplete decl to the USRSet.

2020-02-20 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev accepted this revision. kbobyrev added a comment. This revision is now accepted and ready to land. Let me try to reproduce it in Clang-Rename. If I don't respond by tomorrow, feel free to submit the patch. Comment at: clang/lib/Tooling/Refactoring/Rename/USRFindingAct

[PATCH] D74829: [clang-rename] Add the USR of incomplete decl to the USRSet.

2020-02-19 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: kbobyrev. Herald added subscribers: usaxena95, kadircet, arphaman, jkorous, ilya-biryukov. Herald added a project: clang. This fixes a clangd rename issue, which is missing the reference of an incomplete specialization. Unfortunately, I didn'