Re: [PATCH] Fix avx512{f,vl} shuffles (PR target/87214)

2019-01-27 Thread Uros Bizjak
On 1/26/19, Jakub Jelinek wrote: > Hi! > > The following 4 define_insn shuffle patterns don't have sufficient > conditions. As can be seen even from the way how they transform the > RTL representation into the mask, e.g.: > mask = INTVAL (operands[3]) / 2; > mask |= INTVAL (operands[5]) / 2 <

[PATCH] Fix avx512{f,vl} shuffles (PR target/87214)

2019-01-26 Thread Jakub Jelinek
Hi! The following 4 define_insn shuffle patterns don't have sufficient conditions. As can be seen even from the way how they transform the RTL representation into the mask, e.g.: mask = INTVAL (operands[3]) / 2; mask |= INTVAL (operands[5]) / 2 << 2; mask |= (INTVAL (operands[7]) - 8) / 2 <