[PATCH] D147872: [X86] Correct the scheduling information for AVX-VNNI and AVX512-VNNI instructons.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/X86/X86InstrAVX512.td:12580
                                    EVEX_4V, EVEX_CD8<32, CD8VF>, T8PD,
-                                   Sched<[sched.Folded, sched.ReadAfterFold]>;
+                                   Sched<[sched.Folded, sched.ReadAfterFold,
+                                          sched.ReadAfterFold]>;
----------------
pengfei wrote:
> It's a mystery to me that AVX512 has a lot of 3 ops instructions but not even have one `ReadAfterFold,` in X86InstrAVX512.td
> ```
> grep -rn 'ReadAfterFold,' llvm/lib/Target/X86/X86InstrAVX512.td
> ```
Yeah just from a quick check, vpermt2 vpermi2, and vpternlog are also wrong.

Also interesting, EVEX FMA instructions are wrong but are correct for the VEX versions in X86InstrFMA.td


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147872/new/

https://reviews.llvm.org/D147872