<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><br><pre>At 2016-03-03 15:02:55,ramya@multicorewareinc.com wrote:
># HG changeset patch
># User Ramya Sriraman<ramya@multicorewareinc.com>
># Date 1456985538 -19800
>#      Thu Mar 03 11:42:18 2016 +0530
># Node ID dbccf88be30776f12c7f8c52b9da67d4607abcab
># Parent  9cc9920bf82be1b43efd2a3628e28a3a78ab3b2f
>arm: Implement planecopy_cp NEON
>
>diff -r 9cc9920bf82b -r dbccf88be307 source/common/arm/asm-primitives.cpp
>--- a/source/common/arm/asm-primitives.cpp     Wed Mar 02 17:26:11 2016 +0530
>+++ b/source/common/arm/asm-primitives.cpp     Thu Mar 03 11:42:18 2016 +0530
>@@ -142,6 +142,9 @@
>         p.pu[LUMA_64x48].copy_pp = PFX(blockcopy_pp_64x48_neon);
>         p.pu[LUMA_64x64].copy_pp = PFX(blockcopy_pp_64x64_neon);
> 
>+        // planecopy
>+        p.planecopy_cp = PFX(pixel_planecopy_cp_neon);
>+
>         // sad
>         p.pu[LUMA_8x4].sad    = PFX(pixel_sad_8x4_neon);
>         p.pu[LUMA_8x8].sad    = PFX(pixel_sad_8x8_neon);
>diff -r 9cc9920bf82b -r dbccf88be307 source/common/arm/pixel-util.S
>--- a/source/common/arm/pixel-util.S   Wed Mar 02 17:26:11 2016 +0530
>+++ b/source/common/arm/pixel-util.S   Thu Mar 03 11:42:18 2016 +0530
>@@ -626,3 +626,56 @@
>     pop             {r4, r5}
>     bx              lr
> endfunc
>+
>+function x265_pixel_planecopy_cp_neon
>+    push            {r4, r5, r6}
>+    ldr             r4, [sp, #12]<br>#12 -> #4*3, it means we reserved 3 of Dword in stack <br>
>+    ldr             r5, [sp, #16]<br>#16 -> #4*3 + 4<br>
>+    ldr             r12, [sp, #20]
>+    vdup.8          q2, r12
>+    sub             r5, #1
>+
>+.loop_h:
>+    eor             r6, r6
>+    eor             r12, r12
>+.loop_w:
>+    add             r12, r0, r6<br>Why r0 is constant in loop_w?<br>
>+    vld1.u8         {q0}, [r12]
>+    vshl.u8         q0, q0, q2
>+    add             r12, r2, r6<br>Maybe combo with vld1<br><br>>+    vst1.u8         {q0}, [r12]
>+
>+    add             r6, #16
>+    cmp             r6, r4
>+    blt             .loop_w
>+
<br>Final, this function just call in conditional (X265_DEPTH > 8), so unnecessary 8bits to 8bits copy function <br>
</pre></div>