[PATCH] D101066: [clangd] Dont index deeply nested symbols

2021-04-27 Thread Kadir Cetinkaya via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG4581bf31bb83: [clangd] Dont index deeply nested symbols (authored by kadircet). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION h

[PATCH] D101066: [clangd] Dont index deeply nested symbols

2021-04-23 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet updated this revision to Diff 340091. kadircet added a comment. - Move depth check into a helper and add tests for it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101066/new/ https://reviews.llvm.org/D101066 Files: clang-tools-extra/c

[PATCH] D101066: [clangd] Dont index deeply nested symbols

2021-04-23 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/index/IndexAction.cpp:142 this->Opts.ShouldTraverseDecl = [this](const Decl *D) { + // Many operations performed duri

[PATCH] D101066: [clangd] Dont index deeply nested symbols

2021-04-23 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet updated this revision to Diff 339960. kadircet added a comment. Herald added a subscriber: cfe-commits. - Revert unrelated changes in tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101066/new/ https://reviews.llvm.org/D101066 Files: