[PATCH] D39059: [X86][AVX] Update VCVTx, VMOVNTPx and VROUNDYPx instructions scheduling on btver2

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Target/X86/X86ScheduleBtVer2.td:543
+def : InstRW<[WriteVCVTY], (instregex "VROUNDYP(S|D)r")>;
+def : InstRW<[WriteVCVTY], (instregex "VCVTPS2DQYrr")>;
+
----------------
We need VCVTPS2DQYrr and VCVTTPS2DQYrr


================
Comment at: lib/Target/X86/X86ScheduleBtVer2.td:551
+def : InstRW<[WriteVCVTYLd, ReadAfterLd], (instregex "VROUNDYP(S|D)m")>;
+def : InstRW<[WriteVCVTYLd, ReadAfterLd], (instregex "VCVTPS2DQYrm")>;
+
----------------
We need VCVTPS2DQYrm and VCVTTPS2DQYrm




================
Comment at: lib/Target/X86/X86ScheduleBtVer2.td:564
+}
+def : InstRW<[WriteVCVTPDY], (instregex "VCVTPD2(DQ|PS)Yrr")>;
+
----------------
We need VCVTTPD2DQYrr as well


================
Comment at: lib/Target/X86/X86ScheduleBtVer2.td:570
+}
+def : InstRW<[WriteVCVTPDYLd, ReadAfterLd], (instregex "VCVTPD2(DQ|PS)Yrm")>;
+
----------------
We need VCVTTPD2DQYrm as well


https://reviews.llvm.org/D39059