[PATCH] D29873: [X86][AVX512] Fix operand classes for some AVX512 instructions to keep consistency between VEX/EVEX versions of the same instruction.

[Igor Breger via Phabricator via llvm-commits]

igorb added inline comments.


================
Comment at: lib/Target/X86/X86InstrAVX512.td:5924
+     //             (ins _SrcRC.MemOp:$src),
+                  (ins !cast<Operand>(_SrcRC.Suffix # "mem"):$src),
                   !strconcat(asm,"\t{$src, $dst|$dst, $src}"),
----------------
please remove commented out code
i suggest to add  require info in class X86VectorVTInfo to hold ssmem/sdmem 


https://reviews.llvm.org/D29873