[PATCH] D83768: [clangd] Config: Index.Background

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 14 06:23:14 PDT 2020


sammccall created this revision.
sammccall added a reviewer: kadircet.
Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay, ilya-biryukov.
Herald added a project: clang.

We only support Build/Skip for now, but with 'Load' or similar as an
option for future (load existing shards but don't build new ones).

This requires creating the config for each TU on startup. In LLVM, this
is 4000 occurrences for a total of 800ms on my machine.
But together with caching from D83755 <https://reviews.llvm.org/D83755> it is only 25ms.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D83768

Files:
  clang-tools-extra/clangd/Config.h
  clang-tools-extra/clangd/ConfigCompile.cpp
  clang-tools-extra/clangd/ConfigFragment.h
  clang-tools-extra/clangd/ConfigYAML.cpp
  clang-tools-extra/clangd/index/Background.cpp
  clang-tools-extra/clangd/unittests/BackgroundIndexTests.cpp
  clang-tools-extra/clangd/unittests/ConfigCompileTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83768.277808.patch
Type: text/x-patch
Size: 8448 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200714/a27e5c8a/attachment-0001.bin>


More information about the cfe-commits mailing list