[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