zdtorok marked 8 inline comments as done.
zdtorok added a comment.

Thank you for both of your comments and feedbacks! Also please forgive me for 
my (really) late response. Please have a look at my fixes and the new attached 
diff. Thanks!


================
Comment at: lib/StaticAnalyzer/Checkers/BlockInCriticalSectionChecker.cpp:85
@@ +84,3 @@
+
+void BlockInCriticalSectionChecker::checkPreCall(const CallEvent &Call,
+                                                 CheckerContext &C) const {
----------------
zaks.anna wrote:
> Why is handling of unlock in preCall?
For some reason first I thought it's better for performance reasons - but now 
I've changed it to postCall.


Repository:
  rL LLVM

https://reviews.llvm.org/D21506



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

Reply via email to