[PATCH] D104205: [X86] Schedule-model second (mask) output of GATHER instruction

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 14 00:14:07 PDT 2021


RKSimon added a comment.

What about the avx512 gather ops - are they OK?



================
Comment at: llvm/lib/Target/X86/X86Schedule.td:128
 def WriteMove    : SchedWrite;
+def WriteGatherMaskWriteback : SchedWrite;
 def WriteCopy    : WriteSequence<[WriteLoad, WriteStore]>; // mem->mem copy
----------------
Maybe call this WriteVecMaskedGather to more closely match the other vector load/store classes? Maybe move it to be with the WriteVecMaskedLoad defs as well for clarity.

I'm assuming we don't need separate float/integer classes for this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104205



More information about the llvm-commits mailing list