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

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 9 16:03:42 PDT 2023


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/X86/X86InstrAVX512.td:12589
                                    EVEX_4V, EVEX_CD8<32, CD8VF>, EVEX_B,
-                                   T8PD, Sched<[sched.Folded, sched.ReadAfterFold]>;
+                                   T8PD, Sched<[sched.Folded, sched.ReadAfterFold,
+                                                sched.ReadAfterFold]>;
----------------
LuoYuanke wrote:
> @craig.topper, I don't understand why it needs another sched.ReadAfterFold? Is it because there are 3 operands?
Yes each read is associated with an operand. So we need one for each vector source.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147872



More information about the llvm-commits mailing list