[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