[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