[x265] [PATCH] asm: luma_vpp[8x8] in avx2: improve 701c->387c
Divya Manivannan
divya at multicorewareinc.com
Wed Nov 19 03:19:15 CET 2014
Thanks Chen for the comment. I will make the necessary change.
Regards,
Divya
On Tue, Nov 18, 2014 at 11:25 PM, chen <chenm003 at 163.com> wrote:
> 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
>
>
> _______________________________________________
> 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/07a49f1d/attachment-0001.html>
More information about the x265-devel
mailing list