<div dir="ltr">Definitely more of what I had in mind on the IR. I'd suggest a comment saying what the code generation in the testcases means (at least over the first one).<div><br></div><div>Might want to cc one of the mips guys as a follow up to the commit to make sure that they're happy with it. I believe this is what they want to be actually testing, but there's a chance I'm wrong :)<br><div><br></div><div>-eric</div></div></div><br><div class="gmail_quote">On Mon, Apr 6, 2015 at 3:11 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">New patch attached changing the IR instead.<br>
<br>
On 6 April 2015 at 17:23, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
> 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<br>
> instead of truncating and zero extending the value it should and with i8.<br>
><br>
> -eric<br>
><br>
> On Mon, Apr 6, 2015 at 1:28 PM Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>><br>
> wrote:<br>
>><br>
>> 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<br>
>> > 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>