[x265] [PATCH] asm: luma_vpp[8x8] in avx2: improve 701c->387c

chen chenm003 at 163.com
Tue Nov 18 18:55:18 CET 2014


its right, just a little problem, "movu -> mova"

At 2014-11-18 19:17:53,"Divya Manivannan" <divya at multicorewareinc.com> wrote:
># HG changeset patch
># User Divya Manivannan <divya at multicorewareinc.com>
># Date 1416309303 -19800
>#      Tue Nov 18 16:45:03 2014 +0530
># Node ID 4d6a14960b661796e0935fef347a7d4c87e543a7
># Parent  2f0062f0791b822fa932712a56e6b0a14e976d91
>asm: luma_vpp[8x8] in avx2: improve 701c->387c
>
>diff -r 2f0062f0791b -r 4d6a14960b66 source/common/x86/asm-primitives.cpp
>--- a/source/common/x86/asm-primitives.cpp	Tue Nov 18 11:31:39 2014 +0530
>+++ b/source/common/x86/asm-primitives.cpp	Tue Nov 18 16:45:03 2014 +0530
>@@ -1800,6 +1800,7 @@
> #endif
>         p.luma_hpp[LUMA_4x4] = x265_interp_8tap_horiz_pp_4x4_avx2;
>         p.luma_vpp[LUMA_4x4] = x265_interp_8tap_vert_pp_4x4_avx2;
>+        p.luma_vpp[LUMA_8x8] = x265_interp_8tap_vert_pp_8x8_avx2;
>     }
> #endif // if HIGH_BIT_DEPTH
> }
>diff -r 2f0062f0791b -r 4d6a14960b66 source/common/x86/ipfilter8.asm
>--- a/source/common/x86/ipfilter8.asm	Tue Nov 18 11:31:39 2014 +0530
>+++ b/source/common/x86/ipfilter8.asm	Tue Nov 18 16:45:03 2014 +0530
>@@ -122,6 +122,26 @@
>                   times 8 db 58, -10
>                   times 8 db 4, -1
> 
>+tab_LumaCoeffVer_32: times 16 db 0, 0
>+                     times 16 db 0, 64
>+                     times 16 db 0, 0
>+                     times 16 db 0, 0
>+
>+                     times 16 db -1, 4
>+                     times 16 db -10, 58
>+                     times 16 db 17, -5
>+                     times 16 db 1, 0
>+
>+                     times 16 db -1, 4
>+                     times 16 db -11, 40
>+                     times 16 db 40, -11
>+                     times 16 db 4, -1
>+
>+                     times 16 db 0, 1
>+                     times 16 db -5, 17
>+                     times 16 db 58, -10
>+                     times 16 db 4, -1
>+
> tab_c_64_n64:   times 8 db 64, -64
> 
> 
>@@ -3532,6 +3552,77 @@
> ;-------------------------------------------------------------------------------------------------------------
> FILTER_VER_LUMA_4xN 4, 16, ps
> 
>+%macro PROCESS_LUMA_AVX2_W8_8R 0
>+    movq            xm1, [r0]                       ; m1 = row 0
>+    movq            xm2, [r0 + r1]                  ; m2 = row 1
>+    punpcklbw       xm1, xm2                        ; m1 = [17 07 16 06 15 05 14 04 13 03 12 02 11 01 10 00]
>+    movq            xm3, [r0 + r1 * 2]              ; m3 = row 2
>+    punpcklbw       xm2, xm3                        ; m2 = [27 17 26 16 25 15 24 14 23 13 22 12 21 11 20 10]
>+    vinserti128     m5, m1, xm2, 1                  ; m5 = [27 17 26 16 25 15 24 14 23 13 22 12 21 11 20 10] - [17 07 16 06 15 05 14 04 13 03 12 02 11 01 10 00]
>+    pmaddubsw       m5, [r5]
>+    movq            xm4, [r0 + r4]                  ; m4 = row 3
>+    punpcklbw       xm3, xm4                        ; m3 = [37 27 36 26 35 25 34 24 33 23 32 22 31 21 30 20]
>+    lea             r0, [r0 + r1 * 4]
>+    movq            xm1, [r0]                       ; m1 = row 4
>+    punpcklbw       xm4, xm1                        ; m4 = [47 37 46 36 45 35 44 34 43 33 42 32 41 31 40 30]
>+    vinserti128     m2, m3, xm4, 1                  ; m2 = [47 37 46 36 45 35 44 34 43 33 42 32 41 31 40 30] - [37 27 36 26 35 25 34 24 33 23 32 22 31 21 30 20]
>+    pmaddubsw       m0, m2, [r5 + 1 * mmsize]
>+    paddw           m5, m0
>+    pmaddubsw       m2, [r5]
>+    movq            xm3, [r0 + r1]                  ; m3 = row 5
>+    punpcklbw       xm1, xm3                        ; m1 = [57 47 56 46 55 45 54 44 53 43 52 42 51 41 50 40]
>+    movq            xm4, [r0 + r1 * 2]              ; m4 = row 6
>+    punpcklbw       xm3, xm4                        ; m3 = [67 57 66 56 65 55 64 54 63 53 62 52 61 51 60 50]
>+    vinserti128     m1, m1, xm3, 1                  ; m1 = [67 57 66 56 65 55 64 54 63 53 62 52 61 51 60 50] - [57 47 56 46 55 45 54 44 53 43 52 42 51 41 50 40]
>+    pmaddubsw       m3, m1, [r5 + 2 * mmsize]
>+    paddw           m5, m3
>+    pmaddubsw       m0, m1, [r5 + 1 * mmsize]
>+    paddw           m2, m0
>+    pmaddubsw       m1, [r5]
>+    movq            xm3, [r0 + r4]                  ; m3 = row 7
>+    punpcklbw       xm4, xm3                        ; m4 = [77 67 76 66 75 65 74 64 73 63 72 62 71 61 70 60]
>+    lea             r0, [r0 + r1 * 4]
>+    movq            xm0, [r0]                       ; m0 = row 8
>+    punpcklbw       xm3, xm0                        ; m3 = [87 77 86 76 85 75 84 74 83 73 82 72 81 71 80 70]
>+    vinserti128     m4, m4, xm3, 1                  ; m4 = [87 77 86 76 85 75 84 74 83 73 82 72 81 71 80 70] - [77 67 76 66 75 65 74 64 73 63 72 62 71 61 70 60]
>+    pmaddubsw       m3, m4, [r5 + 3 * mmsize]
>+    paddw           m5, m3
>+    pmaddubsw       m3, m4, [r5 + 2 * mmsize]
>+    paddw           m2, m3
>+    pmaddubsw       m3, m4, [r5 + 1 * mmsize]
>+    paddw           m1, m3
>+    pmaddubsw       m4, [r5]
>+    movq            xm3, [r0 + r1]                  ; m3 = row 9
>+    punpcklbw       xm0, xm3                        ; m0 = [97 87 96 86 95 85 94 84 93 83 92 82 91 81 90 80]
>+    movq            xm6, [r0 + r1 * 2]              ; m6 = row 10
>+    punpcklbw       xm3, xm6                        ; m3 = [A7 97 A6 96 A5 95 A4 94 A3 93 A2 92 A1 91 A0 90]
>+    vinserti128     m0, m0, xm3, 1                  ; m0 = [A7 97 A6 96 A5 95 A4 94 A3 93 A2 92 A1 91 A0 90] - [97 87 96 86 95 85 94 84 93 83 92 82 91 81 90 80]
>+    pmaddubsw       m3, m0, [r5 + 3 * mmsize]
>+    paddw           m2, m3
>+    pmaddubsw       m3, m0, [r5 + 2 * mmsize]
>+    paddw           m1, m3
>+    pmaddubsw       m0, [r5 + 1 * mmsize]
>+    paddw           m4, m0
>+
>+    movq            xm3, [r0 + r4]                  ; m3 = row 11
>+    punpcklbw       xm6, xm3                        ; m6 = [B7 A7 B6 A6 B5 A5 B4 A4 B3 A3 B2 A2 B1 A1 B0 A0]
>+    lea             r0, [r0 + r1 * 4]
>+    movq            xm0, [r0]                       ; m0 = row 12
>+    punpcklbw       xm3, xm0                        ; m3 = [C7 B7 C6 B6 C5 B5 C4 B4 C3 B3 C2 B2 C1 B1 C0 B0]
>+    vinserti128     m6, m6, xm3, 1                  ; m6 = [C7 B7 C6 B6 C5 B5 C4 B4 C3 B3 C2 B2 C1 B1 C0 B0] - [B7 A7 B6 A6 B5 A5 B4 A4 B3 A3 B2 A2 B1 A1 B0 A0]
>+    pmaddubsw       m3, m6, [r5 + 3 * mmsize]
>+    paddw           m1, m3
>+    pmaddubsw       m6, [r5 + 2 * mmsize]
>+    paddw           m4, m6
>+    movq            xm3, [r0 + r1]                  ; m3 = row 13
>+    punpcklbw       xm0, xm3                        ; m0 = [D7 C7 D6 C6 D5 C5 D4 C4 D3 C3 D2 C2 D1 C1 D0 C0]
>+    movq            xm6, [r0 + r1 * 2]              ; m6 = row 14
>+    punpcklbw       xm3, xm6                        ; m3 = [E7 D7 E6 D6 E5 D5 E4 D4 E3 D3 E2 D2 E1 D1 E0 D0]
>+    vinserti128     m0, m0, xm3, 1                  ; m0 = [E7 D7 E6 D6 E5 D5 E4 D4 E3 D3 E2 D2 E1 D1 E0 D0] - [D7 C7 D6 C6 D5 C5 D4 C4 D3 C3 D2 C2 D1 C1 D0 C0]
>+    pmaddubsw       m0, [r5 + 3 * mmsize]
>+    paddw           m4, m0
>+%endmacro
>+
> ;-------------------------------------------------------------------------------------------------------------
> ; void interp_8tap_vert_%3_8x%2(pixel *src, intptr_t srcStride, pixel *dst, intptr_t dstStride, int coeffIdx)
> ;-------------------------------------------------------------------------------------------------------------
>@@ -3601,6 +3692,42 @@
>     RET
> %endmacro
> 
>+INIT_YMM avx2
>+cglobal interp_8tap_vert_pp_8x8, 4, 6, 7
>+    mov             r4d, r4m
>+    shl             r4d, 7
>+
>+%ifdef PIC
>+    lea             r5, [tab_LumaCoeffVer_32]
>+    add             r5, r4
>+%else
>+    lea             r5, [tab_LumaCoeffVer_32 + r4]
>+%endif
>+
>+    lea             r4, [r1 * 3]
>+    sub             r0, r4
>+    PROCESS_LUMA_AVX2_W8_8R
>+    lea             r4, [r3 * 3]
>+    movu            m3, [pw_512]

it isn't error, but aligned address is better, the constant often aligned
>+    pmulhrsw        m5, m3                          ; m5 = word: row 0, row 1
>+    pmulhrsw        m2, m3                          ; m2 = word: row 2, row 3
>+    pmulhrsw        m1, m3                          ; m1 = word: row 4, row 5
>+    pmulhrsw        m4, m3                          ; m4 = word: row 6, row 7
>+    packuswb        m5, m2
>+    packuswb        m1, m4
>+    vextracti128    xm2, m5, 1
>+    vextracti128    xm4, m1, 1
>+    movq            [r2], xm5
>+    movq            [r2 + r3], xm2
>+    movhps          [r2 + r3 * 2], xm5
>+    movhps          [r2 + r4], xm2
>+    lea             r2, [r2 + r3 * 4]
>+    movq            [r2], xm1
>+    movq            [r2 + r3], xm4
>+    movhps          [r2 + r3 * 2], xm1
>+    movhps          [r2 + r4], xm4
>+    RET
>+
> ;-------------------------------------------------------------------------------------------------------------
> ; void interp_8tap_vert_pp_8x4(pixel *src, intptr_t srcStride, pixel *dst, intptr_t dstStride, int coeffIdx)
> ;-------------------------------------------------------------------------------------------------------------
>_______________________________________________
>x265-devel mailing list
>x265-devel at videolan.org
>https://mailman.videolan.org/listinfo/x265-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20141119/9a4d3236/attachment-0001.html>


More information about the x265-devel mailing list