<div dir="ltr">Yes, good catch. Must be:<div><br></div><div>Mask = <span style="font-size:13px;font-family:arial,sans-serif">Mask.trunc(MulWidth);</span><br></div><div><span style="font-size:13px;font-family:arial,sans-serif"><br>
</span></div><div><span style="font-size:13px;font-family:arial,sans-serif">Thanks!</span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-01 21:40 GMT+07:00 Erik Verbruggen <span dir="ltr"><<a href="mailto:erik.verbruggen@me.com" target="_blank">erik.verbruggen@me.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  I also did a patch for PR4917, but yours looks more complete. I have some different tests, and I'll see how they compare against this. Will probably do that tomorrow (CEST).<br>
<br>
  Nitpick: I applied this patch on top of r205314, and I get the following warning:<br>
<br>
  /Users/erik/dev/clang-llvm/llvm-git/lib/Transforms/InstCombine/InstCombineCompares.cpp:2204:9: warning: ignoring return value of function declared with<br>
        warn_unused_result attribute [-Wunused-result]<br>
          Mask.trunc(MulWidth);<br>
          ^~~~~~~~~~ ~~~~~~~~<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2814" target="_blank">http://llvm-reviews.chandlerc.com/D2814</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Thanks,<br>--Serge<br>
</div>