Re: [PATCH] x86: Cast to unsigned short first for _mm_extract_pi16

2021-01-04 Thread Jeff Law via Gcc-patches
On 1/1/21 6:34 AM, H.J. Lu via Gcc-patches wrote: > _mm_extract_pi16 is intrinsic for pextrw, which should be zero-extended, > not sign-extended. > > gcc/ > > PR target/98495 > * config/i386/xmmintrin.h (_mm_extract_pi16): Cast to unsigned > short first. I'd tend to prefer mask

[PATCH] x86: Cast to unsigned short first for _mm_extract_pi16

2021-01-01 Thread H.J. Lu via Gcc-patches
_mm_extract_pi16 is intrinsic for pextrw, which should be zero-extended, not sign-extended. gcc/ PR target/98495 * config/i386/xmmintrin.h (_mm_extract_pi16): Cast to unsigned short first. gcc/testsuite/ PR target/98495 * gcc.target/i386/pr98495-1.c: New