<div dir="ltr"><div class="gmail_default" style="font-family:trebuchet ms,sans-serif;font-size:small">I have sent modified patch.<br><br></div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif;font-size:small">
Thanks,<br></div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif;font-size:small">Dnyaneshwar<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 26, 2014 at 10:39 PM, Steve Borho <span dir="ltr"><<a href="mailto:steve@borho.org" target="_blank">steve@borho.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 08/26, <a href="mailto:dnyaneshwar@multicorewareinc.com">dnyaneshwar@multicorewareinc.com</a> wrote:<br>

> # HG changeset patch<br>
> # User Dnyaneshwar G <<a href="mailto:dnyaneshwar@multicorewareinc.com">dnyaneshwar@multicorewareinc.com</a>><br>
> # Date 1409046621 -19800<br>
> #      Tue Aug 26 15:20:21 2014 +0530<br>
> # Node ID bbd5b3f269b095760d21877e94d67df8bd72f479<br>
> # Parent  5acfb12ec5d17cc700e313fc99248e2408e5967b<br>
> asm: optimize dct4<br>
<br>
</div>You need to provide more detail about how this is an optimization, at<br>
the least the commit message should contain before/after numbers from<br>
the testbench.<br>
<br>
Better if it had mentioned replacing pshufd + pshufhw with pshufb<br>
<div class="HOEnZb"><div class="h5"><br>
> diff -r 5acfb12ec5d1 -r bbd5b3f269b0 source/common/x86/dct8.asm<br>
> --- a/source/common/x86/dct8.asm      Mon Aug 25 17:53:12 2014 +0900<br>
> +++ b/source/common/x86/dct8.asm      Tue Aug 26 15:20:21 2014 +0530<br>
> @@ -30,6 +30,8 @@<br>
><br>
>  SECTION_RODATA 32<br>
><br>
> +dct4_shuf:      db 0, 1, 2, 3, 8, 9, 10, 11, 6, 7, 4, 5, 14, 15, 12, 13<br>
> +<br>
>  tab_dct4:       times 4 dw 64, 64<br>
>                  times 4 dw 83, 36<br>
>                  times 4 dw 64, -64<br>
> @@ -118,16 +120,14 @@<br>
>      movh        m0, [r0 + 0 * r2]<br>
>      movh        m1, [r0 + 1 * r2]<br>
>      punpcklqdq  m0, m1<br>
> -    pshufd      m0, m0, 0xD8<br>
> -    pshufhw     m0, m0, 0xB1<br>
> +    pshufb      m0, [dct4_shuf]<br>
><br>
>      lea         r0, [r0 + 2 * r2]<br>
>      movh        m1, [r0]<br>
>      movh        m2, [r0 + r2]<br>
>      punpcklqdq  m1, m2<br>
> -    pshufd      m1, m1, 0xD8<br>
> -    pshufhw     m1, m1, 0xB1<br>
><br>
> +    pshufb      m1, [dct4_shuf]<br>
>      punpcklqdq  m2, m0, m1<br>
>      punpckhqdq  m0, m1<br>
><br>
> @@ -140,8 +140,7 @@<br>
>      paddd       m3, m7<br>
>      psrad       m3, DCT_SHIFT<br>
>      packssdw    m0, m3<br>
> -    pshufd      m0, m0, 0xD8<br>
> -    pshufhw     m0, m0, 0xB1<br>
> +    pshufb      m0, [dct4_shuf]<br>
>      pmaddwd     m1, m6<br>
>      paddd       m1, m7<br>
>      psrad       m1, DCT_SHIFT<br>
> @@ -149,9 +148,8 @@<br>
>      paddd       m2, m7<br>
>      psrad       m2, DCT_SHIFT<br>
>      packssdw    m1, m2<br>
> -    pshufd      m1, m1, 0xD8<br>
> -    pshufhw     m1, m1, 0xB1<br>
><br>
> +    pshufb      m1, [dct4_shuf]<br>
>      punpcklqdq  m2, m0, m1<br>
>      punpckhqdq  m0, m1<br>
><br>
</div></div><div class="im HOEnZb">> _______________________________________________<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>
</div><span class="HOEnZb"><font color="#888888">--<br>
Steve Borho<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<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>
</div></div></blockquote></div><br></div>