[PATCH] D32679: [X86][AVX-512] Allow EVEX encoded instruction selection when available for mul v8i32.

[Igor Breger via Phabricator via llvm-commits]

igorb added inline comments.


================
Comment at: lib/Target/X86/X86InstrSSE.td:6725
                                  VEX_4V, VEX_WIG;
+let Predicates = [HasAVX] in
   defm VPCMPEQQ : SS48I_binop_rm<0x29, "vpcmpeqq", X86pcmpeq, v2i64, VR128,
----------------
craig.topper wrote:
> This looks like a change for VPCMPEQQ to remove NoVLX.
AVX512 version of  vpcmpeqq  is not equivalent to AVX one.  I think it was disabled by mistake.  

AVX
// Src: (X86pcmpeq:v2i64 VR128:v2i64:$src1, VR128:v2i64:$src2) 
// Dst: (VPCMPEQQrr:v2i64 VR128:v2i64:$src1, VR128:v2i64:$src2)

AVX512
// Src: (X86pcmpeqm:v2i1 VR128X:v2i64:$src1, VR128X:v2i64:$src2) 
// Dst: (VPCMPEQQZ128rr:v2i1 VR128X:v2i64:$src1, VR128X:v2i64:$src2)



https://reviews.llvm.org/D32679