[PATCH] D15464: Add a C++11 ThreadPool implementation in LLVM

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 12 14:58:58 PST 2015


joker.eph closed this revision.
joker.eph marked 2 inline comments as done.
joker.eph added a comment.

r255444


================
Comment at: lib/Support/ThreadPool.cpp:139
@@ +138,3 @@
+ThreadPool::~ThreadPool() {
+  EnableFlag = false;
+  wait();
----------------
tejohnson wrote:
> Is this just for completeness? It doesn't look like this flag is used when ! LLVM_ENABLE_THREADS.
It is to silent a compiler warning about "member variable not used" when compiling with LLVM_ENABLE_THREAD=OFF


http://reviews.llvm.org/D15464





More information about the llvm-commits mailing list