[PATCH] D22408: [clang-rename] add support for overridden functions
Manuel Klimek via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 18 03:12:03 PDT 2016
klimek added inline comments.
================
Comment at: clang-rename/USRFindingAction.cpp:45
@@ +44,3 @@
+namespace {
+// \brief NamedDeclFindingConsumer should delegate finding USRs relevant to
+// given Decl to RelevantUSRFinder.
----------------
Still pondering whether 'relevant' is a good name here...
================
Comment at: clang-rename/USRFindingAction.cpp:72
@@ +71,3 @@
+
+ virtual void run(const MatchFinder::MatchResult &Result) {
+ const auto *VirtualMethod =
----------------
That way, the matchers don't actually get us too much - I'd hoped we already had an hasOverriddenMethod matcher (apparently we don't). It might make sense to implement one.
Then we can implement a matchesUSR matcher, too (which takes a set of USRs), and all those lookups would be cached automatically.
Fine to add a FIXME for this patch though.
================
Comment at: clang-rename/USRFindingAction.cpp:86
@@ +85,3 @@
+
+ void addRelevantUSRs() {
+ // If D is CXXRecordDecl we should add all USRs of its constructors.
----------------
Perhaps call this addUSRsFromOverrideSets or something? addRelevantUSRs doesn't really tell me anything about what it does, and it's currently pretty specific.
https://reviews.llvm.org/D22408
More information about the cfe-commits
mailing list