[x264-devel] [PATCH] arm: Don't assume alignment in mbtree_propagate_list_internal where it isn't provided

Henrik Gramner henrik at gramner.com
Sat Oct 17 14:31:10 CEST 2015


On Sat, Oct 17, 2015 at 1:48 PM, Janne Grunau <janne-x264 at jannau.net> wrote:
> On 2015-10-15 11:50:33 +0300, Martin Storsjö wrote:
>> ---
>>  common/arm/mc-a.S |    4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/common/arm/mc-a.S b/common/arm/mc-a.S
>> index 30d1c1a..7e5f78c 100644
>> --- a/common/arm/mc-a.S
>> +++ b/common/arm/mc-a.S
>> @@ -1829,8 +1829,8 @@ function x264_mbtree_propagate_list_internal_neon
>>  8:
>>      subs            r12, r12,  #8
>>      vld1.16         {q14},  [r1, :128]!      @ propagate_amount
>> -    vld1.16         {q15},  [r2, :128]!      @ lowres_cost
>> -    vld1.16         {q8, q9},  [r0, :128]!
>> +    vld1.16         {q15},  [r2]!            @ lowres_cost
>> +    vld1.16         {q8, q9},  [r0]!
>>      vand            q15, q15, q10
>>      vceq.u16        q1,  q15, q10
>>      vmull.u16       q12, d28, d4
>
> ok
>
> Janne


Applied, thanks.


More information about the x264-devel mailing list