On Tue, Feb 5, 2013 at 1:23 PM, Diego Biurrun <[email protected]> wrote:
> On Fri, Feb 01, 2013 at 06:28:30PM -0500, Daniel Kang wrote:
>>
>> --- a/libavcodec/x86/dsputil.asm
>> +++ b/libavcodec/x86/dsputil.asm
>> @@ -648,3 +650,160 @@ BSWAP32_BUF
>> +
>> +INIT_MMX mmx
>> +cglobal h263_v_loop_filter, 3,5
>> +    movsxdifnidn r1, r1
>> +    movsxdifnidn r2, r2
>> +
>> +INIT_MMX mmx
>> +cglobal h263_h_loop_filter, 3,5,0,32
>> +    movsxdifnidn r1, r1
>> +    movsxdifnidn r2, r2
>
> Is the sign extension still needed after all the ptrdiff_t changes I did?

Yes. You didn't touch h263 loop filter, either that or your changes
haven't been pushed.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to