<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 31, 2013 at 2:24 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, May 31, 2013 at 2:04 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>></span> wrote:<br>
</div></div><div class="gmail_quote"><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
  @jordan_rose, I want this warning.  Not sure about other people<br>
<br>
  @gribozavr, earlier versions of this did trigger on LLVM and Clang.  The warning has been fine-tuned since then to avoid those false positives.<br>
<br>
  Also, I seemed to have messed up the indentation when I wrote the visitors for the first -Wloop-analysis warning and managed to copy the bad indentation over to this change.  I will go fix them.</blockquote><div><br></div>

</div></div><div>Does this find any other bugs (or false positives) in other code you've run it on?</div></div></blockquote><div><br></div><div style> This has found 15-20 bugs so far, with 1-2 false positives.  It is arguable that using (x+=2) in the loop header instead of two separate increments would be clearer for the code.</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
<div>
<a href="http://llvm-reviews.chandlerc.com/D628" target="_blank">http://llvm-reviews.chandlerc.com/D628</a><br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br>
</blockquote></div><br></div></div>