[PATCH] Fix bug in xmmintrin.h

[Eric Christopher]

LGTM. Thanks!

-eric

On Thu, May 22, 2014 at 4:50 PM, Akira Hatanaka <ahatanak at gmail.com> wrote:
> The attached patch changes the last step of _mm_cvtps_pi16, which is a
> function that converts four single precision floating point values into four
> signed 16-bit integers, to use _mm_packs_pi32 instead of _mm_packs_pi16.
>
> <rdar://problem/16873717>
>
> The following program miscompiles without the fix:
>
> #include <xmmintrin.h>
>
> #include <stdio.h>
>
>
> int main(int argc, char **argv)
>
> {
>
>   __m128 test2 = _mm_setr_ps(10.0f, 100.0f, 200.0f, 300.0f);
>
>   __m64 result = _mm_cvtps_pi16(test2);
>
>
>   // wrong value: 0x007f007f0064000a
>
>   // proper value: 0x012c00c80064000a
>
>   printf("%llx\n", result);
>
>
>   return 0;
>
> }
>
>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>