<div dir="ltr">Committed r<span style="color:rgb(0,0,0);white-space:pre-wrap">182984.</span><div><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap">Thanks for fixing this!</span></div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 30, 2013 at 8:09 PM, Jyun-Yan You <span dir="ltr"><<a href="mailto:jyyou@cs.nctu.edu.tw" target="_blank">jyyou@cs.nctu.edu.tw</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">Fixed, please review it.<div class="im"><br><div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Fri, May 31, 2013 at 10:56 AM, Akira Hatanaka <span dir="ltr"><<a href="mailto:ahatanak@gmail.com" target="_blank">ahatanak@gmail.com</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><div>Shouldn't sll use the result of xori (R4), instead of R3? R4 doesn't seem to be used by any other instructions. </div>

<div><br></div><div><div>; CHECK-EB:   sll     $[[R5:[0-9]+]], $[[R3]], 3</div>
</div><div><br></div><div><br></div><div><div>; CHECK-EB:   AtomicLoadAdd8:<br></div></div></div><div>; CHECK-EB:   lw      $[[R0:[0-9]+]], %got(y)</div><div>; CHECK-EB:   addiu   $[[R1:[0-9]+]], $zero, -4</div><div>; CHECK-EB:   and     $[[R2:[0-9]+]], $[[R0]], $[[R1]]</div>


<div>; CHECK-EB:   andi    $[[R3:[0-9]+]], $[[R0]], 3</div><div>; CHECK-EB:   xori    $[[R4:[0-9]+]], $[[R3]], 3</div><div>; CHECK-EB:   sll     $[[R5:[0-9]+]], $[[R3]], 3</div><div>; CHECK-EB:   ori     $[[R6:[0-9]+]], $zero, 255</div>


<div>; CHECK-EB:   sllv    $[[R7:[0-9]+]], $[[R6]], $[[R5]]</div><div>; CHECK-EB:   nor     $[[R8:[0-9]+]], $zero, $[[R7]]</div><div>; CHECK-EB:   sllv    $[[R9:[0-9]+]], $4, $[[R5]]</div><div><br></div><div><div><br></div>


<div>AtomicLoadSub8, AtomicSwap8 and AtomicCmpSwap8 should be fixed too.</div></div><div><br></div></div></blockquote></div><br></div></div></div></div>
</blockquote></div><br></div>