<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div>I suggest just keep one name of sa8d_avx2<br></div><div></div><div id="divNeteaseMailCard"></div><div><br></div>At 2015-12-02 12:31:59,"Dnyaneshwar Gorade" <dnyaneshwar@multicorewareinc.com> wrote:<br> <blockquote id="isReplyContent" style="margin: 0px 0px 0px 0.8ex; padding-left: 1ex; border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid;"><div dir="ltr"><div class="gmail_default" style="font-family: arial,helvetica,sans-serif; font-size: small;">the real function name is sa8d_8x8_avx2 whereas the common function name is sa8d_8x8_avx2_avx2, that's why we got proper call. both are different.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 1, 2015 at 9: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: 0px 0px 0px 0.8ex; padding-left: 1ex; border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid;"><div style="color: rgb(0, 0, 0); line-height: 1.7; font-family: arial; font-size: 14px;"><div><br></div><pre><div><div class="h5"><br>At 2015-12-01 18:58:43,<a href="mailto:dnyaneshwar@multicorewareinc.com" target="_blank">dnyaneshwar@multicorewareinc.com</a> wrote:
># HG changeset patch
># User Dnyaneshwar G <<a href="mailto:dnyaneshwar@multicorewareinc.com" target="_blank">dnyaneshwar@multicorewareinc.com</a>>
># Date 1448962785 -19800
>#      Tue Dec 01 15:09:45 2015 +0530
># Node ID f8b0ce4e9f4092a38d8095961825e734a34f112e
># Parent  e2e507ffe752d6c193a219b242c433bdc55f39f7
>asm: SA8D avx2 asm code for main12
>
>sa8d[  8x8]  4.70x    564.58          2652.82
>sa8d[ 8x16]  4.00x    1358.06         5429.52
>sa8d[16x16]  5.57x    2013.70         11212.47
>sa8d[16x32]  3.90x    5610.47         21883.35
>sa8d[32x32]  5.36x    8274.18         44361.61
>sa8d[32x64]  3.86x    23024.04        88901.80
>sa8d[64x64]  4.35x    45509.79        198165.11
>
>diff -r e2e507ffe752 -r f8b0ce4e9f40 source/common/x86/asm-primitives.cpp
>--- a/source/common/x86/asm-primitives.cpp     Mon Nov 30 11:23:38 2015 +0530
>+++ b/source/common/x86/asm-primitives.cpp     Tue Dec 01 15:09:45 2015 +0530
>@@ -1313,6 +1313,9 @@
>     }
>     if (cpuMask & X265_CPU_AVX2)
>     {
>+#if X265_DEPTH == 12
>+        ASSIGN_SA8D(avx2);
>+#endif
>         <a href="http://p.cu" target="_blank">p.cu</a>[BLOCK_4x4].intra_filter = PFX(intra_filter_4x4_avx2);

>         // TODO: the planecopy_sp is really planecopy_SC now, must be fix it
>diff -r e2e507ffe752 -r f8b0ce4e9f40 source/common/x86/pixel-a.asm
>--- a/source/common/x86/pixel-a.asm    Mon Nov 30 11:23:38 2015 +0530
>+++ b/source/common/x86/pixel-a.asm    Tue Dec 01 15:09:45 2015 +0530
>@@ -6499,6 +6499,1357 @@
> %endif ; !ARCH_X86_64
> %endmacro ; SA8D

>+
>+%if ARCH_X86_64 == 1 && BIT_DEPTH == 12
>+INIT_YMM avx2
>+cglobal sa8d_8x8_avx2
<br></div></div>the really function name is sa8d_8x8_avx2_avx2, we are lucky, below call use correct name</pre></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" rel="noreferrer">https://mailman.videolan.org/listinfo/x265-devel</a><br>
<br></blockquote></div><br></div>
</blockquote></div>