[llvm] [RISCV][VLOPT] Support segmented store instructions (PR #155467)
Min-Yih Hsu via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 3 10:14:06 PDT 2025
================
@@ -1376,6 +1405,83 @@ RISCVVLOptimizer::getMinimumVLForUser(const MachineOperand &UserOp) const {
return VLOp;
}
+/// Return true if MI is an instruction used for assembling registers
+/// for segmented store instructions, namely, RISCVISD::TUPLE_INSERT.
+/// Currently it's lowered to INSERT_SUBREG.
+static bool isTupleInsertInstr(const MachineInstr &MI,
+ const MachineRegisterInfo &MRI) {
+ if (MI.getOpcode() != RISCV::INSERT_SUBREG)
+ return false;
+
+ const TargetRegisterClass *DstRC = MRI.getRegClass(MI.getOperand(0).getReg());
+ // Check whether it was lowered with the correct subreg index.
+ [[maybe_unused]] const TargetRegisterInfo *TRI = MRI.getTargetRegisterInfo();
+ [[maybe_unused]] unsigned SubRegIdx = MI.getOperand(3).getImm();
+ switch (DstRC->getID()) {
+ case RISCV::VRN2M1RegClassID:
+ case RISCV::VRN2M1NoV0RegClassID:
+ case RISCV::VRN3M1RegClassID:
+ case RISCV::VRN3M1NoV0RegClassID:
+ case RISCV::VRN4M1RegClassID:
+ case RISCV::VRN4M1NoV0RegClassID:
+ case RISCV::VRN5M1RegClassID:
+ case RISCV::VRN5M1NoV0RegClassID:
+ case RISCV::VRN6M1RegClassID:
+ case RISCV::VRN6M1NoV0RegClassID:
+ case RISCV::VRN7M1RegClassID:
+ case RISCV::VRN7M1NoV0RegClassID:
+ case RISCV::VRN8M1RegClassID:
+ case RISCV::VRN8M1NoV0RegClassID:
+ assert(TRI->getSubRegIdxSize(SubRegIdx) == RISCV::RVVBitsPerBlock &&
+ "unexpected subreg index for VRM1 sub-register");
+ return true;
+ case RISCV::VRN2M2RegClassID:
+ case RISCV::VRN2M2NoV0RegClassID:
+ case RISCV::VRN3M2RegClassID:
+ case RISCV::VRN3M2NoV0RegClassID:
+ case RISCV::VRN4M2RegClassID:
+ case RISCV::VRN4M2NoV0RegClassID:
+ assert(TRI->getSubRegIdxSize(SubRegIdx) == RISCV::RVVBitsPerBlock * 2 &&
+ "unexpected subreg index for VRM2 sub-register");
+ return true;
+ case RISCV::VRN2M4RegClassID:
+ case RISCV::VRN2M4NoV0RegClassID:
+ assert(TRI->getSubRegIdxSize(SubRegIdx) == RISCV::RVVBitsPerBlock * 4 &&
+ "unexpected subreg index for VRM4 sub-register");
+ return true;
+ default:
+ return false;
+ }
+}
+
+static bool isSegmentedStoreInstr(const MachineInstr &MI) {
----------------
mshockwave wrote:
yeah we probably should, there are still plenty of space in TSFlags. I'll do that in a follow-up patch.
https://github.com/llvm/llvm-project/pull/155467
More information about the llvm-commits
mailing list