[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
> 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"
More information about the cfe-commits