Janne Grunau <[email protected]> writes:

>> diff --git a/libavcodec/arm/vp3dsp_neon.S b/libavcodec/arm/vp3dsp_neon.S
>> index 8d22d00..2a9b25f 100644
>> --- a/libavcodec/arm/vp3dsp_neon.S
>> +++ b/libavcodec/arm/vp3dsp_neon.S
>> @@ -116,9 +116,8 @@ function vp3_idct_start_neon
>>      vadd.s16        q1,  q8,  q12
>>      vsub.s16        q8,  q8,  q12
>>      vld1.64         {d28-d31}, [r2,:128]!
>> -endfunc
>>  
>> -function vp3_idct_core_neon
>> +vp3_idct_core_neon:
>>      vmull.s16       q2,  d18, xC1S7     // (ip[1] * C1) << 16
>>      vmull.s16       q3,  d19, xC1S7
>>      vmull.s16       q4,  d2,  xC4S4     // ((ip[0] + ip[4]) * C4) << 16
>
> not mentioned in the commit message, it looked unrelated before I saw
> the macro changes

I guess I could have been clearer about that.

[...]

> I would wish for something simpler.

If I could think of a simpler way, I'd use it.  The problem is that gas
only allows constants with the ldr reg, =val syntax, so I have to use
macros to record the offsets as they are used and dump them at the end
of the function.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to