<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 17, 2013 at 2:43 AM,  <span dir="ltr"><<a href="mailto:yuvaraj@multicorewareinc.com" target="_blank">yuvaraj@multicorewareinc.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Yuvaraj Venkatesh <<a href="mailto:yuvaraj@multicorewareinc.com">yuvaraj@multicorewareinc.com</a>><br>
# Date 1381995774 -19800<br>
#      Thu Oct 17 13:12:54 2013 +0530<br>
# Node ID 6f7ce9050675df305f39c16f7aa6e43767eecb0e<br>
# Parent  1d6b3626f1b3d4b8f8f181e66eaaa649f90bac02<br>
pixel: fix the hash mismatch due to convert32to16_shr<br></blockquote><div><br></div><div>I'm holding this patch hostage until there's a test function for these convert primitives :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

diff -r 1d6b3626f1b3 -r 6f7ce9050675 source/common/vec/pixel-sse3.cpp<br>
--- a/source/common/vec/pixel-sse3.cpp  Wed Oct 16 20:38:44 2013 -0500<br>
+++ b/source/common/vec/pixel-sse3.cpp  Thu Oct 17 13:12:54 2013 +0530<br>
@@ -44,7 +44,7 @@<br>
         im32 = _mm_loadu_si128((__m128i const*)org);<br>
         im32 = _mm_sra_epi32(_mm_add_epi32(im32, round), _mm_cvtsi32_si128(shift));<br>
         im16 = _mm_packs_epi32(im32, im32);<br>
-        _mm_storeu_si128((__m128i*)dst, im16);<br>
+        _mm_storel_epi64((__m128i*)dst, im16);<br>
<br>
         org += 4;<br>
         dst += 4;<br>
@@ -636,7 +636,7 @@<br>
 namespace x265 {<br>
 void Setup_Vec_PixelPrimitives_sse3(EncoderPrimitives &p)<br>
 {<br>
-    //p.cvt32to16_shr = convert32to16_shr;<br>
+    p.cvt32to16_shr = convert32to16_shr;<br>
     p.cvt16to32_shl = convert16to32_shl;<br>
     p.cvt16to16_shl = convert16to16_shl;<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">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Steve Borho
</div></div>