[PATCH] D26799: [X86][AVX512] Instructions fixups

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 17 19:04:27 PST 2016


craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86InstrAVX512.td:6431
 
-defm VCVTDQ2PD : avx512_cvtdq2pd<0xE6, "vcvtdq2pd", sint_to_fp, X86cvtdq2pd>, XS,
-                                EVEX_CD8<32, CD8VH>;
+defm VCVTDQ2PD : avx512_cvtdq2pd<0xE6, "vcvtdq2pd", sint_to_fp, X86cvtdq2pd, i128mem>,
+                                XS, EVEX_CD8<32, CD8VH>;
----------------
Shouldn't this be i64mem too? The only difference between VCVTDQ2PD and VCVTUDQ2PD is that one is signed and the other is unsigned. They should have the same load size.


Repository:
  rL LLVM

https://reviews.llvm.org/D26799





More information about the llvm-commits mailing list