LGTM. Thanks! -eric
On Thu, May 22, 2014 at 4:50 PM, Akira Hatanaka <[email protected]> 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 > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
