hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land.
Thanks! ================ Comment at: clang-tools-extra/clangd/XRefs.cpp:1294 + OverriddenMethods.insert(ID); + getOverriddenMethods(Base, OverriddenMethods); + } ---------------- The recursive will be infinite if we have a cycle, but I guess clang will never generate a broken AST like that. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95852/new/ https://reviews.llvm.org/D95852 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
