On Tue, Jun 11, 2013 at 10:37 PM,  <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</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">Joerg Sonnenberger <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>> writes:<br>
>> Do you mean something like this?<br>
><br>
> Yes, with a few more (non-trivial) test vectors.<br>
<br>
</div>Here's a version with some more interesting tests. I've also opted to<br>
treat __builtin_clz(0) as non-const as per Richard's suggestion.</blockquote><div><br></div><div>Don't return false from VisitCallExpr without emitting a diagnostic (see the comment at the top of the file) -- replace the</div>
<div><br></div><div>  if (!Val)</div><div>    return false;</div><div><br></div><div>with</div><div><br></div><div>  if (!Val)</div><div>    return Error(E);</div><div><br></div><div>Otherwise LGTM, thanks!</div></div>