<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div><br></div><pre><br>At 2015-04-28 09:05:06,dtyx265@gmail.com wrote:
># HG changeset patch
># User David T Yuen <a href="mailto:dtyx265@gmail.com>># Date 1430182995 25200># Node ID 31b76bd430a47411f7b2ebaa7cfbb44e25c5ff60># Parent  68a13226d586b335c02cade9311e093f0149c42a>asm: interp_8tap_horiz pp and ps sse2>>This replaces c code and covers>">dtyx265@gmail.com>
># Date 1430182995 25200
># Node ID 31b76bd430a47411f7b2ebaa7cfbb44e25c5ff60
># Parent  68a13226d586b335c02cade9311e093f0149c42a
>asm: interp_8tap_horiz pp and ps sse2
>
>This replaces c code and covers
>
</a>
>diff -r 68a13226d586 -r 31b76bd430a4 source/common/x86/asm-primitives.cpp
>--- a/source/common/x86/asm-primitives.cpp     Sat Apr 25 01:39:55 2015 -0500
>+++ b/source/common/x86/asm-primitives.cpp     Mon Apr 27 18:03:15 2015 -0700
>@@ -1340,6 +1340,10 @@
>         CHROMA_420_VSP_FILTERS(_sse2);
>         CHROMA_422_VSP_FILTERS(_sse2);
>         CHROMA_444_VSP_FILTERS(_sse2);
>+        ALL_LUMA_PU(luma_hpp, interp_8tap_horiz_pp, sse2);
>+        p.pu[LUMA_4x4].luma_hpp = x265_interp_8tap_horiz_pp_4x4_sse2;
>+        ALL_LUMA_PU(luma_hps, interp_8tap_horiz_ps, sse2);
>+        p.pu[LUMA_4x4].luma_hps = x265_interp_8tap_horiz_ps_4x4_sse2;

>         //p.frameInitLowres = x265_frame_init_lowres_core_mmx2;
>         p.frameInitLowres = x265_frame_init_lowres_core_sse2;
>diff -r 68a13226d586 -r 31b76bd430a4 source/common/x86/ipfilter8.asm
>--- a/source/common/x86/ipfilter8.asm  Sat Apr 25 01:39:55 2015 -0500
>+++ b/source/common/x86/ipfilter8.asm  Mon Apr 27 18:03:15 2015 -0700
>@@ -160,6 +160,11 @@
>                  db  -1, 4, -11, 40,  40, -11, 4, -1
>                  db   0, 1, -5,  17,  58, -10, 4, -1

>+tabw_LumaCoeff:  dw   0, 0,  0,  64,  0,   0,  0,  0
>+                 dw  -1, 4, -10, 58,  17, -5,  1,  0
>+                 dw  -1, 4, -11, 40,  40, -11, 4, -1
>+                 dw   0, 1, -5,  17,  58, -10, 4, -1
>+
> tab_LumaCoeffV: times 4 dw 0, 0
>                 times 4 dw 0, 64
>                 times 4 dw 0, 0
>@@ -825,6 +830,230 @@
>     IPFILTER_CHROMA_W_sse3 48, 64
>     IPFILTER_CHROMA_W_sse3 64, 16

>+%macro FILTER_H8_W8_sse2 0
>+    movh        m1, [r0 + r5 - 3]<br>offset field (-3) can avoid by preare on r0<br>
>+    movh        m4, [r0 + r5 - 3 + 1]
>+    punpcklbw   m1, m6
>+    punpcklbw   m4, m6
>+    movh        m5, [r0 + r5 - 3 + 2]
>+    movh        m0, [r0 + r5 - 3 + 3]
>+    punpcklbw   m5, m6
>+    punpcklbw   m0, m6
>+    pmaddwd     m1, m3
>+    pmaddwd     m4, m3
>+    pmaddwd     m5, m3
>+    pmaddwd     m0, m3
>+    packssdw    m1, m4
>+    packssdw    m5, m0
>+    pshuflw     m4, m1, q2301
>+    pshufhw     m4, m4, q2301
>+    pshuflw     m0, m5, q2301
>+    pshufhw     m0, m0, q2301
>+    paddw       m1, m4
>+    paddw       m5, m0
>+    psrldq      m1, 2
>+    psrldq      m5, 2
>+    pshufd      m1, m1, q3120
>+    pshufd      m5, m5, q3120
>+    punpcklqdq  m1, m5<br>m1 valid in here<br>
>+    movh        m7, [r0 + r5 - 3 + 4]
>+    movh        m4, [r0 + r5 - 3 + 5]
>+    punpcklbw   m7, m6
>+    punpcklbw   m4, m6
>+    movh        m5, [r0 + r5 - 3 + 6]
>+    movh        m0, [r0 + r5 - 3 + 7]
>+    punpcklbw   m5, m6
>+    punpcklbw   m0, m6
>+    pmaddwd     m7, m3
>+    pmaddwd     m4, m3
>+    pmaddwd     m5, m3
>+    pmaddwd     m0, m3
>+    packssdw    m7, m4
>+    packssdw    m5, m0
>+    pshuflw     m4, m7, q2301
>+    pshufhw     m4, m4, q2301
>+    pshuflw     m0, m5, q2301
>+    pshufhw     m0, m0, q2301
>+    paddw       m7, m4
>+    paddw       m5, m0
>+    psrldq      m7, 2
>+    psrldq      m5, 2
>+    pshufd      m7, m7, q3120
>+    pshufd      m5, m5, q3120
>+    punpcklqdq  m7, m5
>+    pshuflw     m4, m1, q2301<br>m1 used in here, long distance<br>
>+    pshufhw     m4, m4, q2301
>+    pshuflw     m0, m7, q2301
>+    pshufhw     m0, m0, q2301
>+    paddw       m1, m4
>+    paddw       m7, m0
>+    psrldq      m1, 2
>+    psrldq      m7, 2
>+    pshufd      m1, m1, q3120
>+    pshufd      m7, m7, q3120
>+    punpcklqdq  m1, m7
>+%endmacro
>+
<br>>+;----------------------------------------------------------------------------------------------------------------------------
>+; void interp_8tap_horiz_%3_%1x%2(pixel *src, intptr_t srcStride, pixel *dst, intptr_t dstStride, int coeffIdx, int isRowExt)
>+;----------------------------------------------------------------------------------------------------------------------------
>+%macro IPFILTER_LUMA_sse2 3
>+INIT_XMM sse2
>+cglobal interp_8tap_horiz_%3_%1x%2, 4,7,8
>+
>+    mov       r4d, r4m
>+    add       r4d, r4d
>+    pxor      m6, m6
>+%ifdef PIC
>+    lea       r6, [tabw_LumaCoeff]
>+    mova      m3, [r6 + r4 * 8]
>+%else
>+    mova      m3, [tabw_LumaCoeff + r4 * 8]
>+%endif
>+
>+    mov       r4d, %2
>+%ifidn %3, pp
>+    mova      m2, [pw_32]
>+%else
>+    mova      m2, [pw_2000]
>+    add       r3d, r3d
>+    cmp       r5m, byte 0
>+    je        .loopH
>+    lea       r6, [r1 + 2 * r1]
>+    sub       r0d, r6d
>+    add       r4d, 7
>+%endif
>+
>+.loopH:
>+    xor       r5d, r5d
>+%rep %1 / 8<br>use %rep here, why you need physical register r5?<br>
>+    FILTER_H8_W8_sse2
>+  %ifidn %3, pp
>+    paddw     m1, m2
>+    psraw     m1, 6
>+    packuswb  m1, m1
>+    movh      [r2 + r5], m1
>+  %else
>+    psubw     m1, m2
>+    movu      [r2 + 2 * r5], m1
>+  %endif
>+    add       r5d, 8
>+%endrep
>+
>+%rep (%1 % 8) / 4
>+    FILTER_H8_W4_sse2
>+  %ifidn %3, pp
>+    paddw     m1, m2
>+    psraw     m1, 6
>+    packuswb  m1, m1
>+    movd      [r2 + r5], m1
>+  %else
>+    psubw     m1, m2
>+    movh      [r2 + 2 * r5], m1
>+  %endif
>+%endrep
>+
>+    add       r0d, r1d
>+    add       r2d, r3d
>+
>+    dec       r4d
>+    jnz       .loopH
>+<br>no extra blank line here<br>
>+    RET
>+
>+%endmacro
</pre></div>