[PATCH] D54290: [libcxx] Provide thread annotations for shared_mutex

Petr Hosek via Phabricator reviews at reviews.llvm.org
Fri Nov 9 11:54:34 PST 2018


phosek marked an inline comment as done.
phosek added inline comments.


================
Comment at: libcxx/include/shared_mutex:164
     // Exclusive ownership
-    void lock(); // blocking
-    bool try_lock();
-    void unlock();
+    void lock() _LIBCPP_THREAD_SAFETY_ANNOTATION(acquire_capability()); // blocking
+    bool try_lock() _LIBCPP_THREAD_SAFETY_ANNOTATION(try_acquire_capability(true));
----------------
ldionne wrote:
> Just making sure; those are only needed on the declarations?
Yes


Repository:
  rCXX libc++

https://reviews.llvm.org/D54290





More information about the libcxx-commits mailing list