This revision was automatically updated to reflect the committed changes.
Closed by commit rG3c5745cb1f3c: [clangd] Make background index thread count
calculation clearer (authored by sammccall).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82352/ne
sammccall marked an inline comment as done.
sammccall added inline comments.
Comment at: clang-tools-extra/clangd/index/Background.h:140
+ // In production an explicit value is passed.
+ size_t ThreadPoolSize = 4,
std::function OnProgress = nullptr);
kadircet accepted this revision.
kadircet added a comment.
This revision is now accepted and ready to land.
Thanks LGTM!
Comment at: clang-tools-extra/clangd/index/Background.h:140
+ // In production an explicit value is passed.
+ size_t ThreadPoolSize = 4,
std
aganea added a comment.
LGTM. Thanks for reverting.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82352/new/
https://reviews.llvm.org/D82352
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
sammccall created this revision.
sammccall added reviewers: kadircet, aganea.
Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay,
ilya-biryukov.
Herald added a project: clang.
This confusion was inadvertently introduced in a change to the
heavyweight_hardware_concurrency