[FFmpeg-cvslog] r16822 - trunk/libavcodec/arm/dsputil_arm_s.S

[Måns Rullgård]

Benoit Fouet <benoit.fouet at free.fr> writes:

> On 01/27/2009 05:06 PM, mru wrote:
>> Author: mru
>> Date: Tue Jan 27 17:06:47 2009
>> New Revision: 16822
>>
>> Log:
>> ARM: replace jump tables with conditional branches
>>
>> Modified:
>>    trunk/libavcodec/arm/dsputil_arm_s.S
>>
>> Modified: trunk/libavcodec/arm/dsputil_arm_s.S
>> ==============================================================================
>> --- trunk/libavcodec/arm/dsputil_arm_s.S	Tue Jan 27 17:06:44 2009	(r16821)
>> +++ trunk/libavcodec/arm/dsputil_arm_s.S	Tue Jan 27 17:06:47 2009	(r16822)
>> @@ -90,6 +90,17 @@ function ff_prefetch_arm, export=1
>>          add \Rd1, \Rn1, \Rd1, lsr #1
>>  .endm
>>  
>> +.macro  JMP_ALIGN tmp, reg, label1, label2, label3
>> +        ands \tmp, \reg, #3
>> +        bic  \reg, \reg, #3
>> +        beq  1f
>> +        subs \tmp, \tmp, #1
>> +        beq  2f
>> +        subs \tmp, \tmp, #1
>> +        beq  3f
>> +        b    4f
>> +.endm
>> +
>
> shouldn't that be using labelX at some point ?

I changed my mind and forgot to remove them.  Thanks for keeping an
eye on things though.

-- 
M?ns Rullg?rd
mans at mansr.com