[PATCH] D64682: [clangd] Don't rebuild background index until we indexed one TU per thread.

Kadir Cetinkaya via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 15 03:05:16 PDT 2019


kadircet accepted this revision.
kadircet added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang-tools-extra/clangd/index/BackgroundRebuild.h:51
+      SwapIndex *Target, FileSymbols *Source,
+      unsigned Threads = llvm::heavyweight_hardware_concurrency())
+      : TUsBeforeFirstBuild(Threads), Target(Target), Source(Source) {}
----------------
let's not have a default here since it will get out-dated if we decide to change default for BackgroundIndex's constructor.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64682/new/

https://reviews.llvm.org/D64682





More information about the llvm-commits mailing list