[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