I reverted the patch in r161736. I'll investigate this on Monday.<br><br>On Saturday, August 11, 2012, Michael Gottesman  wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

This patch caused an execution correctness error for test MultiSource/Applications/JM/ldecod/ldecod. The specific error that occurs is that the output image differs from the expected result image beyond a certain tolerance. The error occurs on armv7 thumb O0 g and armv7 thumb O3.<br>



<br>
On Aug 8, 2012, at 5:18 PM, Evan Cheng <<a>evan.cheng@apple.com</a>> wrote:<br>
<br>
><br>
> On Aug 7, 2012, at 8:59 AM, Arnold Schwaighofer <<a>arnolds@codeaurora.org</a>> wrote:<br>
><br>
>> Updated patch. Removed unused RLo, RHi. Added test cases (forgot to add them to the index before).<br>
>><br>
>> Okay to commit?<br>
><br>
> Yes.<br>
><br>
> Evan<br>
><br>
>><br>
>> On 8/5/2012 5:28 PM, Arnold Schwaighofer wrote:<br>
>>> Yes, you are right - that must have been an oversight. I'll remove them<br>
>>> before commiting.<br>
>>> Thanks for catching this.<br>
>>><br>
>>><br>
>>>> +class AsMla1I64<bits<7>  opcod, dag oops, dag iops, InstrItinClass itin,<br>
>>>> +             string opc, string asm, list<dag>  pattern><br>
>>>> +  : AsMul1I<opcod, oops, iops, itin, opc, asm, pattern>  {<br>
>>>> +  bits<4>  RdLo;<br>
>>>> +  bits<4>  RdHi;<br>
>>>> +  bits<4>  Rm;<br>
>>>> +  bits<4>  Rn;<br>
>>>> +  bits<4>  RLo;<br>
>>>> +  bits<4>  RHi;<br>
>>>> +  let Inst{19-16} = RdHi;<br>
>>>> +  let Inst{15-12} = RdLo;<br>
>>>> +  let Inst{11-8}  = Rm;<br>
>>>> +  let Inst{3-0}   = Rn;<br>
>>>> +}<br>
>>>><br>
>>>> Hi Arnold,<br>
>>>><br>
>>>> Here it seems that RLo and RHi are useless in this class, right? If it's<br>
>>>> no, could you please point me where they are being referenced?<br>
>>>><br>
>>>> Thanks,<br>
>>>> -Jiangning<br>
>>>><br>
>>>> 在 2012-8-2，下午10:57，Arnold Schwaighofer<<a>arnolds@codeaurora.org</a>><br>
>>>> 写道：<br>
>>>><br>
>>>>> Patch to implement UMLAL/SMLAL Instructions for the ARM Architecture<br>
>>>>><br>
>>>>> This patch corrects the definition of umlal/smlal instructions and adds<br>
>>>>> support for matching them to the ARM dag combiner.<br>
>>>>><br>
>>>>><br>
>>>>><br>
>>>>> This patch got lost on my end. Sorry.<br>
>>>>><br>
>>>>> Is it okay to commit?<br>
>>>>><br>
>>>>> Prior discussion:<br>
>>>>> <a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120507/142351.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120507/142351.html</a><br>
>>>>> <a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120416/141153.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120416/141153.html</a><br>
>>>>><br>
>>>>> Bug:<br>
>>>>> <a href="http://llvm.org/bugs/show_bug.cgi?id=12213" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=12213</a><br>
>>>>><br>
>>>>>> I understand Anton has not gotten back to you. That's fine, I don't<br>
>>>>>> think<br>
>>>>>> you need to wait for him. However, it looks like Arnold has some<br>
>>>>>> concerns<br>
>>>>>> about the patch? Please address his concerns first.<br>
>>>>>><br>
>>>>>> Once you have addressed Arnold's concerns, he can commit it for you.<br>
>>>>>> Arnold has commit privilege.<br>
>>>>>><br>
>>>>>> Evan<br>
>><br>
>><br>
>> <yinmas_mla2.patch>_______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a>llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a>llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.</a></blockquote>