[x265] [PATCH 2 of 3] fix pixel_ssim_end4, the 3rd is dword

Jason Garrett-Glaser jason at x264.com
Fri Apr 25 07:50:19 CEST 2014


On Thu, Apr 24, 2014 at 9:30 PM, Jason Garrett-Glaser <jason at x264.com> wrote:
> On Thu, Apr 24, 2014 at 8:02 PM, Min Chen <chenm003 at 163.com> wrote:
>> # HG changeset patch
>> # User Min Chen <chenm003 at 163.com>
>> # Date 1398394896 -28800
>> # Node ID 89875c3fdaf689c4e0d7488264dbda6eed266326
>> # Parent  9505724b11bc944eaaefd35d396f92f4c2c3f88f
>> fix pixel_ssim_end4, the 3rd is dword
>>
>> diff -r 9505724b11bc -r 89875c3fdaf6 source/common/x86/pixel-util8.asm
>> --- a/source/common/x86/pixel-util8.asm Fri Apr 25 11:01:12 2014 +0800
>> +++ b/source/common/x86/pixel-util8.asm Fri Apr 25 11:01:36 2014 +0800
>> @@ -1981,7 +1981,8 @@
>>  ;-----------------------------------------------------------------------------
>>  ; float pixel_ssim_end( int sum0[5][4], int sum1[5][4], int width )
>>  ;-----------------------------------------------------------------------------
>> -cglobal pixel_ssim_end4, 3,3,7
>> +cglobal pixel_ssim_end4, 2,3,7
>> +    mov       r2d, r2m
>
> You might find movifnidn useful.
>
> Jason

Oh, I see, you're doing zero extension. You could change the argument
to intptr_t for that, maybe?

Jason


More information about the x265-devel mailing list