[PATCH] D33415: [clangd] Replaced WorkerRequest with std::function...

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 23 04:56:56 PDT 2017

ilya-biryukov added inline comments.

Comment at: clangd/ClangdServer.cpp:102
   } // unlock Mutex
+  RequestCV.notify_one();
krasimir wrote:
> Why did this get out?
"The notifying thread does not need to hold the lock on the same mutex as the one held by the waiting thread(s); in fact doing so is a pessimization, since the notified thread would immediately block again, waiting for the notifying thread to release the lock"
(from: http://en.cppreference.com/w/cpp/thread/condition_variable/notify_one)


More information about the cfe-commits mailing list