[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