dgoldman marked an inline comment as done.
dgoldman added inline comments.
================
Comment at: clang-tools-extra/clangd/FindTarget.cpp:716
+ void VisitObjCImplementationDecl(const ObjCImplementationDecl *OIMD) {
+ if (const auto *CI = OIMD->getClassInterface())
+ Refs.push_back(ReferenceLoc{NestedNameSpecifierLoc(),
----------------
kadircet wrote:
> we don't have the null check in other places, what's the significance here?
Removed, doesn't look like it's needed.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152720/new/
https://reviews.llvm.org/D152720
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
