[PATCH] D155654: [RISCV] Add SchedRead for Merge operands on MASK Pseudos
Wang Pengcheng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 21 03:00:18 PDT 2023
wangpc added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td:2172
+ defvar ReadWritesMask = InsertAtIdx1<ReadWrites, ReadVMergeOp_MX_E>.l;
+ defm _VV : VPseudoBinaryEmul<m.vrclass, m.vrclass, emul.vrclass, m,
+ emul, ReadWrites, ReadWritesMask,
----------------
wangpc wrote:
> We should add some multiclasses to define `Sched`s (just like existed multiclasses for load/store, e.g., `VLESchedMask`, ) instead of passing `SchedRead/SchedWrite`s to these multiclasses which are used to define pseudos. They make the code messy, I think.
> We can seperate these multiclasses into other patches to make this patch easy to review. I am pleased to help you with this.
`multiclass`->`class`
Please see D155932, feedback is welcome.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155654/new/
https://reviews.llvm.org/D155654
More information about the llvm-commits
mailing list