[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
Tue Mar 21 10:28:26 PDT 2023


michaelmaitland accepted this revision.
michaelmaitland added a comment.
This revision is now accepted and ready to land.

LGTM. As we discussed offline, today `UpperBound` refers to the largest LMUL supported. This should be changed in a future patch. I propose that it should be removed from MxList and its meaning should be tied the SchedWrite with the worst case behaviour. Now that we are supporting SchedWrite by LMUL + SEW or just LMUL depending on the instruction, the specific scheduler model can decide which LMUL or LMUL + SEW corresponds to the worst case behaviour. This means no assumption is made on whether the Largest LMUL or smallest SEW behaves the worst.


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