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

[Aaron Puchert via Phabricator via llvm-commits]

aaronpuchert added a comment.
Herald added a project: All.

Shouldn't the attributes be on `shared_mutex` and `shared_timed_mutex`, respectively, instead of the internal `__shared_mutex_base`? Note that the analysis is strictly intraprocedural and doesn't do "inlining" <https://clang.llvm.org/docs/ThreadSafetyAnalysis.html#no-inlining>.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54290/new/

https://reviews.llvm.org/D54290