<div dir="ltr">You're right. We came up with the usual case, but not the normal case. Kyle is going to add a check for the arguments as well :)<div><br></div><div>-eric</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Mar 1, 2016 at 2:30 PM Hal Finkel <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">hfinkel added a subscriber: hfinkel.<br>
hfinkel requested changes to this revision.<br>
hfinkel added a reviewer: hfinkel.<br>
hfinkel added a comment.<br>
This revision now requires changes to proceed.<br>
<br>
I don't think this is right. RLWINM starts by taking two copies of the 32-bit amount, one copy in the lower word and one copy in the upper word and rotates the result, Then it composes the mask. The mask is non-zero in the upper bits if MB >= ME.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D17781" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17781</a><br>
<br>
<br>
<br>
</blockquote></div>