<div dir="ltr">I brought up the same question last year.  By including this in the CFG builder, the other analysis that use the CFG can use the tautological results for better diagnostics.<div><br></div><div><a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20131028/091857.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20131028/091857.html</a><br>

</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 25, 2014 at 12:30 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 dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5">On Mon, Aug 25, 2014 at 9:19 AM, Anders Rönnholm <span dir="ltr"><<a href="mailto:Anders.Ronnholm@evidente.se" target="_blank">Anders.Ronnholm@evidente.se</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I have made a new bitwise comparison always true check for relational comparisons i'd like to get reviewed.<br>
<br>
e.g<br>
if ((x & 64) < 65) // bitwise comparison always evaluates to true</blockquote><div><br></div></div></div><div>I'm very surprised to see this being implemented in Analysis/CFG.cpp; this belongs in Sema/SemaChecking.cpp. It makes no sense to me to couple this check to building a CFG.</div>

</div></div></div>
</blockquote></div><br></div>