delesley accepted this revision.
delesley added a comment.

This looks good to me.  Thanks for the patch.



================
Comment at: lib/Analysis/ThreadSafety.cpp:932
+      // We're relocking the underlying mutexes. Warn on double locking.
+      if (FSet.findLock(FactMan, UnderCp))
+        Handler.handleDoubleLock(DiagKind, UnderCp.toString(), entry.loc());
----------------
Minor nit.  Use curly braces on the if, to match the else.


Repository:
  rC Clang

https://reviews.llvm.org/D49885



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

Reply via email to