sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land.
Great, thanks! ================ Comment at: clang-tools-extra/clangd/FindSymbols.cpp:176 +std::string getSymbolName(ASTContext &Ctx, const NamedDecl &ND) { + if (const auto *Container = dyn_cast<ObjCContainerDecl>(&ND)) + return printObjCContainer(*Container); ---------------- can we motivate this with a comment? // print X rather than Y Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96612/new/ https://reviews.llvm.org/D96612 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits