<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div>Thanks you idea, I also plan move it to conditionally compile.</div><div><br></div><div>btw: we asm use BIT_DEPTH and C use X265_DEPTH<br></div><div></div><div id="divNeteaseMailCard"></div><div><br></div>At 2015-07-15 15:09:07,"Deepthi Nandakumar" <deepthi@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>Min,<br><br></div>These should be disabled conditionally (if X265_DEPTH==12). Btw, I think in your earlier Main12 fix patches, it should X265_DEPTH, and not BIT_DEPTH? Please check. <br><br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 15, 2015 at 6:36 AM, Min 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;"># HG changeset patch<br>
# User Min Chen <<a href="mailto:chenm003@163.com">chenm003@163.com</a>><br>
# Date 1436916622 25200<br>
# Node ID 8abf1d4209bbd721a60b3f94d9553dec88e2e8fe<br>
# Parent  39d3e3941f865084c5b559f97f6178d26c7751c2<br>
asm: disable Main12 error functions<br>
---<br>
 source/common/x86/asm-primitives.cpp |    9 ++++-----<br>
 1 files changed, 4 insertions(+), 5 deletions(-)<br>
<br>
diff -r 39d3e3941f86 -r 8abf1d4209bb source/common/x86/asm-primitives.cpp<br>
--- a/source/common/x86/asm-primitives.cpp      Tue Jul 14 16:29:46 2015 -0700<br>
+++ b/source/common/x86/asm-primitives.cpp      Tue Jul 14 16:30:22 2015 -0700<br>
@@ -917,7 +917,7 @@<br>
         p.pu[LUMA_4x4].satd = <a href="http://p.cu" target="_blank" rel="noreferrer">p.cu</a>[BLOCK_4x4].sa8d = PFX(pixel_satd_4x4_mmx2);<br>
         ALL_LUMA_PU(satd, pixel_satd, sse2);<br>
<br>
-        ASSIGN_SA8D(sse2);<br>
+        //ASSIGN_SA8D(sse2);<br>
         LUMA_PIXELSUB(sse2);<br>
         CHROMA_420_PIXELSUB_PS(sse2);<br>
         CHROMA_422_PIXELSUB_PS(sse2);<br>
@@ -958,7 +958,7 @@<br>
         ALL_LUMA_TU_S(calcresidual, getResidual, sse2);<br>
         ALL_LUMA_TU_S(transpose, transpose, sse2);<br>
<br>
-        ALL_LUMA_TU_S(intra_pred[PLANAR_IDX], intra_pred_planar, sse2);<br>
+        //ALL_LUMA_TU_S(intra_pred[PLANAR_IDX], intra_pred_planar, sse2);<br>
         ALL_LUMA_TU_S(intra_pred[DC_IDX], intra_pred_dc, sse2);<br>
<br>
         <a href="http://p.cu" target="_blank" rel="noreferrer">p.cu</a>[BLOCK_4x4].intra_pred[2] = PFX(intra_pred_ang4_2_sse2);<br>
<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>
</blockquote></div><br></div>
</blockquote></div>