<div dir="ltr">Ok, I will do it for the future blocks.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Dec 5, 2013 at 8:08 PM, chen <span dir="ltr"><<a href="mailto:chenm003@163.com" target="_blank">chenm003@163.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="line-height:1.7;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,<a href="mailto:murugan@multicorewareinc.com" target="_blank">murugan@multicorewareinc.com</a> wrote:<div>
<div class="h5"><br>># HG changeset patch<br>># User Murugan Vairavel <<a href="mailto:murugan@multicorewareinc.com" target="_blank">murugan@multicorewareinc.com</a>><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></div></div>>_______________________________________________<br>>x265-devel mailing list<br>
><a href="mailto:x265-devel@videolan.org" target="_blank">x265-devel@videolan.org</a><br>><a href="https://mailman.videolan.org/listinfo/x265-devel" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</div></div><br>_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">With Regards,<div><br></div><div>Murugan. V</div><div>+919659287478</div></div>
</div>