[all-commits] [llvm/llvm-project] 907efd: [clangd] Cleanup FuzzyFindRequest serialization an...

Kirill Bobyrev via All-commits all-commits at lists.llvm.org
Wed Jul 21 05:51:49 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 907efdf95da567c314e2c9b9df67187756eed5ca
      https://github.com/llvm/llvm-project/commit/907efdf95da567c314e2c9b9df67187756eed5ca
  Author: Kirill Bobyrev <kbobyrev at google.com>
  Date:   2021-07-21 (Wed, 21 Jul 2021)

  Changed paths:
    M clang-tools-extra/clangd/benchmarks/IndexBenchmark.cpp
    M clang-tools-extra/clangd/index/Index.cpp
    M clang-tools-extra/clangd/index/dex/Iterator.cpp

  Log Message:
  -----------
  [clangd] Cleanup FuzzyFindRequest serialization and dex benchmark

* Due to the LLVM's JSON library changes (?), FuzzyFindRequest serialization is
  no longer valid since arrays are serialized as llvm::json::Array already.
  Hence, current implementation creates a nested array.
* YAML format is no longer the default, mention this for the benchmark.
* FIXME is no longer relevant. I ran benchmarks that showed no improvement with
  priority_queue years ago.

Reviewed By: sammccall

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




More information about the All-commits mailing list