<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><DIV>the code is right and applyed.</DIV>
<DIV>next time, I suggest you reduce XMM register, the number more than 6 will save/restore with stack, it is slower<BR><BR>At 2013-12-05 18:28:34,murugan@multicorewareinc.com wrote:<BR>># HG changeset patch<BR>># User Murugan Vairavel <murugan@multicorewareinc.com><BR>># Date 1386239288 -19800<BR>>#      Thu Dec 05 15:58:08 2013 +0530<BR>># Node ID 4f17d311db3e0a58dac851533e5db16b874d6bc2<BR>># Parent  589f43958b63b96c3192967494555e5af9c2cde6<BR>>asm: 10bpp cpde for pixel_sub_2xN<BR>><BR>>diff -r 589f43958b63 -r 4f17d311db3e source/common/x86/asm-primitives.cpp<BR>>--- a/source/common/x86/asm-primitives.cpp  Thu Dec 05 15:20:30 2013 +0530<BR>>+++ b/source/common/x86/asm-primitives.cpp    Thu Dec 05 15:58:08 2013 +0530<BR>>@@ -570,6 +570,9 @@<BR>> <BR>>         p.cvt32to16_shr = x265_cvt32to16_shr_sse2;<BR>>         p.cvt16to32_shl = x265_cvt16to32_shl_sse2;<BR>>+<BR>>+    p.chroma[X265_CSP_I420].sub_ps[LUMA_4x8] = x265_pixel_sub_ps_2x4_sse2;<BR>>+    p.chroma[X265_CSP_I420].sub_ps[LUMA_4x16] = x265_pixel_sub_ps_2x8_sse2;<BR>>     }<BR>>     if (cpuMask & X265_CPU_SSSE3)<BR>>     {<BR>>diff -r 589f43958b63 -r 4f17d311db3e source/common/x86/pixel-util.h<BR>>--- a/source/common/x86/pixel-util.h        Thu Dec 05 15:20:30 2013 +0530<BR>>+++ b/source/common/x86/pixel-util.h  Thu Dec 05 15:58:08 2013 +0530<BR>>@@ -121,6 +121,7 @@<BR>> <BR>> CHROMA_PIXELSUB_DEF(_sse4);<BR>> LUMA_PIXELSUB_DEF(_sse4);<BR>>+CHROMA_PIXELSUB_DEF(_sse2);<BR>> <BR>> #define SETUP_LUMA_PIXELVAR_FUNC(W, H, cpu) \<BR>>     uint64_t x265_pixel_var_ ## W ## x ## H ## cpu(pixel * pix, intptr_t pixstride);<BR>>diff -r 589f43958b63 -r 4f17d311db3e source/common/x86/pixel-util8.asm<BR>>--- a/source/common/x86/pixel-util8.asm  Thu Dec 05 15:20:30 2013 +0530<BR>>+++ b/source/common/x86/pixel-util8.asm       Thu Dec 05 15:58:08 2013 +0530<BR>>@@ -2072,6 +2072,33 @@<BR>> ;-----------------------------------------------------------------------------<BR>> ; void pixel_sub_ps_c_2x4(int16_t *dest, intptr_t destride, pixel *src0, pixel *src1, intptr_t srcstride0, intptr_t srcstride1);<BR>> ;-----------------------------------------------------------------------------<BR>>+%if HIGH_BIT_DEPTH<BR>>+INIT_XMM sse2<BR>>+cglobal pixel_sub_ps_2x4, 6, 6, 8, dest, deststride, src0, src1, srcstride0, srcstride1<BR>>+    add      r1,    r1<BR>>+    add      r4,    r4<BR>>+    add      r5,    r5<BR>>+    movd     m0,    [r2]<BR>>+    movd     m1,    [r3]<BR>>+    movd     m2,    [r2 + r4]<BR>>+    movd     m3,    [r3 + r5]<BR>>+    movd     m4,    [r2 + 2 * r4]<BR>>+    movd     m5,    [r3 + 2 * r5]<BR>>+    lea      r2,    [r2 + 2 * r4]<BR>>+    lea      r3,    [r3 + 2 * r5]<BR>>+    movd     m6,    [r2 + r4]<BR>>+    movd     m7,    [r3 + r5]<BR>>+    psubw    m0,    m1<BR>>+    psubw    m2,    m3<BR>>+    psubw    m4,    m5<BR>>+    psubw    m6,    m7<BR>>+<BR>>+    movd     [r0],           m0<BR>>+    movd     [r0 + r1],      m2<BR>>+    movd     [r0 + 2 * r1],  m4<BR>>+    lea      r0,             [r0 + 2 * r1]<BR>>+    movd     [r0 + r1],      m6<BR>>+%else<BR>> INIT_XMM sse4<BR>> %if ARCH_X86_64<BR>>     cglobal pixel_sub_ps_2x4, 6, 8, 0<BR>>@@ -2156,12 +2183,63 @@<BR>> sub      t0d,      t1d<BR>> <BR>> mov      [r0 + 2], t0w<BR>>-<BR>>+%endif<BR>> RET<BR>> <BR>> ;-----------------------------------------------------------------------------<BR>> ; void pixel_sub_ps_c_2x8(int16_t *dest, intptr_t destride, pixel *src0, pixel *src1, intptr_t srcstride0, intptr_t srcstride1);<BR>> ;-----------------------------------------------------------------------------<BR>>+%if HIGH_BIT_DEPTH<BR>>+INIT_XMM sse2<BR>>+cglobal pixel_sub_ps_2x8, 6, 6, 8, dest, deststride, src0, src1, srcstride0, srcstride1<BR>>+    add      r1,    r1<BR>>+    add      r4,    r4<BR>>+    add      r5,    r5<BR>>+    movd     m0,    [r2]<BR>>+    movd     m1,    [r3]<BR>>+    movd     m2,    [r2 + r4]<BR>>+    movd     m3,    [r3 + r5]<BR>>+    movd     m4,    [r2 + 2 * r4]<BR>>+    movd     m5,    [r3 + 2 * r5]<BR>>+    lea      r2,    [r2 + 2 * r4]<BR>>+    lea      r3,    [r3 + 2 * r5]<BR>>+    movd     m6,    [r2 + r4]<BR>>+    movd     m7,    [r3 + r5]<BR>>+    psubw    m0,    m1<BR>>+    psubw    m2,    m3<BR>>+    psubw    m4,    m5<BR>>+    psubw    m6,    m7<BR>>+<BR>>+    movd     [r0],           m0<BR>>+    movd     [r0 + r1],      m2<BR>>+    movd     [r0 + 2 * r1],  m4<BR>>+    lea      r0,             [r0 + 2 * r1]<BR>>+    movd     [r0 + r1],      m6<BR>>+<BR>>+    movd     m0,    [r2 + 2 * r4]<BR>>+    movd     m1,    [r3 + 2 * r5]<BR>>+    lea      r2,    [r2 + 2 * r4]<BR>>+    lea      r3,    [r3 + 2 * r5]<BR>>+    movd     m2,    [r2 + r4]<BR>>+    movd     m3,    [r3 + r5]<BR>>+    movd     m4,    [r2 + 2 * r4]<BR>>+    movd     m5,    [r3 + 2 * r5]<BR>>+    lea      r2,    [r2 + 2 * r4]<BR>>+    lea      r3,    [r3 + 2 * r5]<BR>>+    movd     m6,    [r2 + r4]<BR>>+    movd     m7,    [r3 + r5]<BR>>+    psubw    m0,    m1<BR>>+    psubw    m2,    m3<BR>>+    psubw    m4,    m5<BR>>+    psubw    m6,    m7<BR>>+<BR>>+    movd     [r0 + 2 * r1],  m0<BR>>+    lea      r0,             [r0 + 2 * r1]<BR>>+    movd     [r0 + r1],      m2<BR>>+    movd     [r0 + 2 * r1],  m4<BR>>+    lea      r0,             [r0 + 2 * r1]<BR>>+    movd     [r0 + r1],      m6<BR>>+%else<BR>> INIT_XMM sse4<BR>> %if ARCH_X86_64<BR>>     cglobal pixel_sub_ps_2x8, 6, 8, 0<BR>>@@ -2308,7 +2386,7 @@<BR>>     sub      t0d,      t1d<BR>> <BR>>     mov      [r0 + 2], t0w<BR>>-<BR>>+%endif<BR>> RET<BR>> <BR>> ;-----------------------------------------------------------------------------<BR>>_______________________________________________<BR>>x265-devel mailing list<BR>>x265-devel@videolan.org<BR>>https://mailman.videolan.org/listinfo/x265-devel<BR></DIV></div>