Hello Jakub! On 28 Dec 10:10, Jakub Jelinek wrote: > Hi! > > These insns don't really need AVX512BW in any way themselves, only their > masked variants might need it for reloading of the mask register, but that > should be covered in builtins.def, doesn't need duplication in sse.md. > For non-masked it causes ICEs, because the builtins properly aren't guarded > with AVX512BW, but the insns incorrectly require that. > > Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for > trunk? Your patch is OK.
> > 2017-12-28 Jakub Jelinek <ja...@redhat.com> > > PR target/83604 > * config/i386/sse.md (VI248_VLBW): Rename to ... > (VI248_AVX512VL): ... this. Don't guard V32HI with TARGET_AVX512BW. > (vpshrd_<mode><mask_name>, vpshld_<mode><mask_name>, > vpshrdv_<mode>, vpshrdv_<mode>_mask, vpshrdv_<mode>_maskz, > vpshrdv_<mode>_maskz_1, vpshldv_<mode>, vpshldv_<mode>_mask, > vpshldv_<mode>_maskz, vpshldv_<mode>_maskz_1): Use VI248_AVX512VL > mode iterator instead of VI248_VLBW. > > * gcc.target/i386/pr83604.c: New test. -- Thanks, K
