aaronpuchert planned changes to this revision.
aaronpuchert added a comment.

This doesn't work with loops yet:

  void relockLoop() {
    RelockableExclusiveMutexLock scope(&mu);
  
    while (b) {
      scope.Unlock(); // releasing mutex 'scope' that was not held
      scope.Lock();   // acquiring mutex 'mu' that is already held
    }
  }

There should be no warnings here — this code is fine.


Repository:
  rC Clang

https://reviews.llvm.org/D51187



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

Reply via email to