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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to