[PATCH] D65708: [NFC][DirectoryWatchedTests] Unlocking mutexes before signaling condition variable.

2019-08-05 Thread Puyan Lotfi via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL367968: [NFC][DirectoryWatchedTests] Unlocks mutexes before signaling condition variable (authored by zer0, committed by

[PATCH] D65708: [NFC][DirectoryWatchedTests] Unlocking mutexes before signaling condition variable.

2019-08-05 Thread Jan Korous via Phabricator via cfe-commits
jkorous added a comment. LGTM. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65708/new/ https://reviews.llvm.org/D65708 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D65708: [NFC][DirectoryWatchedTests] Unlocking mutexes before signaling condition variable.

2019-08-03 Thread Puyan Lotfi via Phabricator via cfe-commits
plotfi created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. plotfi added reviewers: compnerd, jkorous. Herald added a subscriber: dexonsmith. This should not affect actual behavior, but should pessimize the threading less by avoiding the situation where: