[PATCH] D81332: Thread safety analysis: Support deferring locks

Aaron Puchert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 8 07:38:25 PDT 2020


aaronpuchert added a comment.

In D81332#2079489 <https://reviews.llvm.org/D81332#2079489>, @aaron.ballman wrote:

> LGTM! Should we update the public docs for this change as well? Specifically, I am wondering if we should update https://clang.llvm.org/docs/ThreadSafetyAnalysis.html#mutexheader


Yes, some existing functionality seems undocumented as well (like adopting locks), which is why I was planning a bigger change.

My plan is basically to go through the test cases (which seem to be quite complete in terms of what we intend to support) and see what deserves mention in the official docs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81332





More information about the cfe-commits mailing list