[PATCH] D111039: [clangd] Include refs of base method in refs for derived method.

Utkarsh Saxena via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 5 01:43:16 PDT 2021


usaxena95 marked an inline comment as done.
usaxena95 added inline comments.


================
Comment at: clang-tools-extra/clangd/XRefs.cpp:1397
+              OverriddenBy.Subjects.insert(ID);
               getOverriddenMethods(CMD, OverriddenMethods);
             }
----------------
kadircet wrote:
> note that there's a behaviour change here as well. not sure how often it matters in practice, but previously we would still get the overriden methods for a symbol even if we fail to generate symbolid for it. let's keep it the same.
Hmm. You are right. Even in `getOverriddenMethods` we find the symbolID *optionally* and we always traverse the complete type hierarchy. Let's do the same here for consistency.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111039/new/

https://reviews.llvm.org/D111039



More information about the cfe-commits mailing list