This revision was automatically updated to reflect the committed changes.
kadircet marked an inline comment as done.
Closed by commit rG81967b4fa77a: [clangd] Don't trim xrefs references if
we overran the limit (authored by kadircet).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTI
sammccall added inline comments.
Comment at: clang-tools-extra/clangd/XRefs.cpp:1388
+ TargetsInMainFile.insert(ID);
+ // Not all symbols can be referenced from outside (e.g. function-locals).
+ // TODO: we could skip TU-scoped symbols here (e.g. static functions)
sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.
Comment at: clang-tools-extra/clangd/XRefs.cpp:1388
+ TargetsInMainFile.insert(ID);
+ // Not all symbols can be referenced from outside (e.g. function-loc
kadircet created this revision.
kadircet added a reviewer: sammccall.
Herald added subscribers: usaxena95, arphaman.
kadircet requested review of this revision.
Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov.
Herald added a project: clang-tools-extra.
This preserves all the results
