<div dir="ltr">Thanks, committed r209489.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 22, 2014 at 5:32 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">LGTM. Thanks!<br>
<br>
-eric<br>
<div><div class="h5"><br>
On Thu, May 22, 2014 at 4:50 PM, Akira Hatanaka <<a href="mailto:ahatanak@gmail.com">ahatanak@gmail.com</a>> wrote:<br>
> The attached patch changes the last step of _mm_cvtps_pi16, which is a<br>
> function that converts four single precision floating point values into four<br>
> signed 16-bit integers, to use _mm_packs_pi32 instead of _mm_packs_pi16.<br>
><br>
> <rdar://problem/16873717><br>
><br>
> The following program miscompiles without the fix:<br>
><br>
> #include <xmmintrin.h><br>
><br>
> #include <stdio.h><br>
><br>
><br>
> int main(int argc, char **argv)<br>
><br>
> {<br>
><br>
>   __m128 test2 = _mm_setr_ps(10.0f, 100.0f, 200.0f, 300.0f);<br>
><br>
>   __m64 result = _mm_cvtps_pi16(test2);<br>
><br>
><br>
>   // wrong value: 0x007f007f0064000a<br>
><br>
>   // proper value: 0x012c00c80064000a<br>
><br>
>   printf("%llx\n", result);<br>
><br>
><br>
>   return 0;<br>
><br>
> }<br>
><br>
><br>
><br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</blockquote></div><br></div>