<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div>At 2014-04-25 13:50:19,"Jason Garrett-Glaser" <jason@x264.com> wrote:<br>>On Thu, Apr 24, 2014 at 9:30 PM, Jason Garrett-Glaser <jason@x264.com> wrote:<br>>> On Thu, Apr 24, 2014 at 8:02 PM, Min Chen <chenm003@163.com> wrote:<br>>>> # HG changeset patch<br>>>> # User Min Chen <chenm003@163.com><br>>>> # Date 1398394896 -28800<br>>>> # Node ID 89875c3fdaf689c4e0d7488264dbda6eed266326<br>>>> # Parent  9505724b11bc944eaaefd35d396f92f4c2c3f88f<br>>>> fix pixel_ssim_end4, the 3rd is dword<br>>>><br>>>> diff -r 9505724b11bc -r 89875c3fdaf6 source/common/x86/pixel-util8.asm<br>>>> --- a/source/common/x86/pixel-util8.asm Fri Apr 25 11:01:12 2014 +0800<br>>>> +++ b/source/common/x86/pixel-util8.asm Fri Apr 25 11:01:36 2014 +0800<br>>>> @@ -1981,7 +1981,8 @@<br>>>>  ;-----------------------------------------------------------------------------<br>>>>  ; float pixel_ssim_end( int sum0[5][4], int sum1[5][4], int width )<br>>>>  ;-----------------------------------------------------------------------------<br>>>> -cglobal pixel_ssim_end4, 3,3,7<br>>>> +cglobal pixel_ssim_end4, 2,3,7<br>>>> +    mov       r2d, r2m<br>>><br>>> You might find movifnidn useful.<br>>><br>>> Jason<br>><br>>Oh, I see, you're doing zero extension. You could change the argument<br>>to intptr_t for that, maybe?<br>><br>Thanks</div>
<div>The code copy from x264, so I do the zero extendsion on it.</div>
<div> </div></div>