<div dir="ltr">Ah right. That's... both obvious and non-obvious without some thought. :)<br><br>That said, can you fix the testcases to do the right thing here? I think instead of truncating and zero extending the value it should and with i8.<div><br></div><div>-eric</div></div><br><div class="gmail_quote">On Mon, Apr 6, 2015 at 1:28 PM Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 6 April 2015 at 16:23, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
> Hi Rafael,<br>
><br>
> The changes to the mips fast isel test are worrying. From looking at the<br>
> testcase it's materializing a constant 1 and you have it now materializing a<br>
> constant -1 (extended) into the register.<br>
><br>
<br>
I think that is a bug fix, no? "i1 1" is -1.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div>