a.sidorin accepted this revision.
a.sidorin added a comment.
This revision is now accepted and ready to land.

Just a minor nit.



================
Comment at: lib/StaticAnalyzer/Core/BugReporter.cpp:3332
+
+    if (Visited.count(Blk)) {
+      // We've encountered a loop. We won't see anything new here anymore.
----------------
The check for visited can be placed into the loop below so we won't even add 
such blocks to the worklist.


https://reviews.llvm.org/D35673



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

Reply via email to