[llvm] [RISCV] Simplify FindRegWithEncoding in copyPhysRegVector. NFC (PR #89001)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 16 16:35:11 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-risc-v
Author: Craig Topper (topperc)
<details>
<summary>Changes</summary>
Instead of searching all encodings, we can convert the encoding back to a register and use getMatchingSuperReg.
---
Full diff: https://github.com/llvm/llvm-project/pull/89001.diff
1 Files Affected:
- (modified) llvm/lib/Target/RISCV/RISCVInstrInfo.cpp (+6-9)
``````````diff
diff --git a/llvm/lib/Target/RISCV/RISCVInstrInfo.cpp b/llvm/lib/Target/RISCV/RISCVInstrInfo.cpp
index 14b5cbea71722f..8331fc0b8c3024 100644
--- a/llvm/lib/Target/RISCV/RISCVInstrInfo.cpp
+++ b/llvm/lib/Target/RISCV/RISCVInstrInfo.cpp
@@ -361,15 +361,12 @@ void RISCVInstrInfo::copyPhysRegVector(
return {RISCVII::LMUL_1, RISCV::VRRegClass, RISCV::VMV1R_V,
RISCV::PseudoVMV_V_V_M1, RISCV::PseudoVMV_V_I_M1};
};
- auto FindRegWithEncoding = [&TRI](const TargetRegisterClass &RegClass,
- uint16_t Encoding) {
- ArrayRef<MCPhysReg> Regs = RegClass.getRegisters();
- const auto *FoundReg = llvm::find_if(Regs, [&](MCPhysReg Reg) {
- return TRI->getEncodingValue(Reg) == Encoding;
- });
- // We should be always able to find one valid register.
- assert(FoundReg != Regs.end());
- return *FoundReg;
+ auto FindRegWithEncoding = [TRI](const TargetRegisterClass &RegClass,
+ uint16_t Encoding) {
+ MCRegister Reg = RISCV::V0 + Encoding;
+ if (&RegClass == &RISCV::VRRegClass)
+ return Reg;
+ return TRI->getMatchingSuperReg(Reg, RISCV::sub_vrm1_0, &RegClass);
};
while (I != NumRegs) {
// For non-segment copying, we only do this once as the registers are always
``````````
</details>
https://github.com/llvm/llvm-project/pull/89001
More information about the llvm-commits
mailing list