[PATCH] D61724: [clangd] Use AsyncTaskRunner in BackgroundIndex instead of std::thread

Brennan Vincent via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 13 08:26:54 PDT 2019


umanwizard added inline comments.


================
Comment at: clang-tools-extra/trunk/clangd/index/Background.cpp:164
   stop();
-  for (auto &Thread : ThreadPool)
-    Thread.join();
+  ThreadPool.wait();
 }
----------------
This is already called in `~AsyncTaskRunner`.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61724





More information about the llvm-commits mailing list