kadircet accepted this revision.
kadircet added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: clang-tools-extra/clangd/index/Background.cpp:152
+ for (unsigned I = 1; I <= ThreadPoolSize; ++I) {
+ ThreadPool.runAsync("background-worker-" + llvm::Twine(I),
+ [this] { run(); });
----------------
NIT: why not count from zero :P
================
Comment at: clang-tools-extra/clangd/index/Background.cpp:164
stop();
- for (auto &Thread : ThreadPool)
- Thread.join();
+ ThreadPool.wait();
}
----------------
Destructor of `AsyncTaskRunner` already does that
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61724/new/
https://reviews.llvm.org/D61724
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
