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