[PATCH] D116043: [clangd] Don't trim xrefs references if we overran the limit

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 20 08:01:49 PST 2021


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 we've processed already rather than
throwing them away in the end.
It has some performance implications on the edge cases, in the worst case we
might issue 1 relations and 2 xrefs requests in extra to deduce `HasMore`
correctly.

Fixes https://github.com/clangd/clangd/issues/204.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D116043

Files:
  clang-tools-extra/clangd/XRefs.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116043.395454.patch
Type: text/x-patch
Size: 7224 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20211220/82bf5c8d/attachment.bin>


More information about the cfe-commits mailing list