[PATCH] D49885: Thread safety analysis: Allow relockable scopes

Aaron Puchert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 13 18:59:08 PDT 2018


aaronpuchert updated this revision to Diff 160505.
aaronpuchert added a comment.

Fix crash. The problem was that ACQUIRES with arguments did not no longer have (only) `this` as argument, hence our assumption that we would have only ScopedLockableFactEntry's was false. So now we lock a bit closer which kind of ACQUIRES we actually have.


Repository:
  rC Clang

https://reviews.llvm.org/D49885

Files:
  lib/Analysis/ThreadSafety.cpp
  test/SemaCXX/warn-thread-safety-analysis.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49885.160505.patch
Type: text/x-patch
Size: 12443 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180814/2cb6c102/attachment.bin>


More information about the cfe-commits mailing list