[PATCH] D144925: [RISCV][NFC] Replace the pseudos for instructions that depend on lmul with variants that encode the SEW into the name

Michael Maitland via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 15 10:26:49 PDT 2023


michaelmaitland requested changes to this revision.
michaelmaitland added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td:2337
+        defvar ReadVFSqrtV_MX = !cast<SchedRead>("ReadVFSqrtV_" # mx # "_E" # e);
+        defvar suffix = mx # "_E" # e;
+
----------------
nitinjohnraj wrote:
> michaelmaitland wrote:
> > Use `suffix` in `WriteVFSqrtV_MX` and `ReadVFSqrtV_MX`. Also need to add `_E` to  `WriteVFSqrtV_MX` and `ReadVFSqrtV_MX`
> Thanks, will do!
`defvar WriteVFSqrtV_MX_E` and `defvar WriteVFSqrtV_MX_E`


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td:2350
 
+// TODO: Should I make this SEW aware?
 multiclass VPseudoVRCP_V {
----------------
craig.topper wrote:
> michaelmaitland wrote:
> > Where did you land on making this SEW aware?
> I think the answer is no.
If this does not need to be SEW aware, should we remove the TODO?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144925



More information about the llvm-commits mailing list