On 07/31/2012 10:37 AM, Diego Biurrun wrote:
> On Mon, Jul 30, 2012 at 01:48:09PM -0400, Justin Ruggles wrote:
>>
>> --- a/libavcodec/x86/fmtconvert.asm
>> +++ b/libavcodec/x86/fmtconvert.asm
>> @@ -27,18 +27,20 @@ SECTION_TEXT
>> -%if WIN64
>> +%if ARCH_X86_32
>> +    SPLATD    m0, mulm
>> +%else
>> +    %if WIN64
>>      SWAP 0, 2
>> -%elif ARCH_X86_32
>> -    movss   m0, mulm
>> +    %endif
>> +    SPLATD    m0
>>  %endif
>> -    SPLATD  m0
> 
> %elif WIN64

That would not be correct unless I also duplicate the "SPLATD m0" for a
separate %else case.

-Justin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to