[all-commits] [llvm/llvm-project] 81967b: [clangd] Don't trim xrefs references if we overran...

kadir ├žetinkaya via All-commits all-commits at lists.llvm.org
Mon Dec 20 08:23:02 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 81967b4fa77ab5fc9840f272cb4fab1814df11e7
      https://github.com/llvm/llvm-project/commit/81967b4fa77ab5fc9840f272cb4fab1814df11e7
  Author: Kadir Cetinkaya <kadircet at google.com>
  Date:   2021-12-20 (Mon, 20 Dec 2021)

  Changed paths:
    M clang-tools-extra/clangd/XRefs.cpp

  Log Message:
  -----------
  [clangd] Don't trim xrefs references if we overran the limit

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.

Differential Revision: https://reviews.llvm.org/D116043




More information about the All-commits mailing list