<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt">On Thu, Nov 15, 2012 at 2:50 PM, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de" target="_blank">joerg@britannica.bec.de</a>></span> wrote:<br>
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Nov 15, 2012 at 02:02:54PM -0800, Richard Smith wrote:<br>
> On Thu, Nov 15, 2012 at 1:57 PM, Joerg Sonnenberger<br>
> <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>> wrote:<br>
> > On Wed, Nov 14, 2012 at 10:50:24PM -0000, Richard Trieu wrote:<br>
> >> Author: rtrieu<br>
> >> Date: Wed Nov 14 16:50:24 2012<br>
> >> New Revision: 167992<br>
> >><br>
> >> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=167992&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=167992&view=rev</a><br>
> >> Log:<br>
> >> Improve -Wtautological-constant-out-of-range-compare by taking into account<br>
> >> type conversion between integers.  This allows the warning to be more accurate.<br>
> >><br>
> >> Also, turned the warning off in an analyzer test.  The relavent test cases<br>
> >> are covered by the tests in Sema.<br>
> ><br>
> > This gives invalid warnings now. Example:<br>
> ><br>
> > typedef enum {<br>
> >  PT_DESCENDING=-1,<br>
> >  PT_ASCENDING=1<br>
> > } pt_direction_t;<br>
> ><br>
> > int<br>
> > ptree_iterate(pt_direction_t direction)<br>
> > {<br>
> >   return (direction == PT_ASCENDING) ? 1 : 0;<br>
> > }<br>
><br>
> Was this not fixed by r168023?<br>
<br>
</div>I have r168047 here.<br>
<span class="HOEnZb"><font color="#888888"><br>
Joerg<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">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><div>This looks like a bad calculation of the integer range.  I'll take a look into it.</div></div>