[llvm] [RISCV] Use non-VP select in gather->strided load combine. NFCish (PR #157070)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 5 03:47:48 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-risc-v
Author: Luke Lau (lukel97)
<details>
<summary>Changes</summary>
Similar to #<!-- -->157068, remove our reliance on trivial VP intrinsics by using a regular select instruction and letting RISCVVLOptimizer take care of VL.
---
Full diff: https://github.com/llvm/llvm-project/pull/157070.diff
1 Files Affected:
- (modified) llvm/lib/Target/RISCV/RISCVISelLowering.cpp (+3-3)
``````````diff
diff --git a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
index f8ec1be1fd8d6..27442841704aa 100644
--- a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
+++ b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
@@ -20550,9 +20550,9 @@ SDValue RISCVTargetLowering::PerformDAGCombine(SDNode *N,
VT, DL, MGN->getChain(), BasePtr,
DAG.getSignedConstant(StepNumerator, DL, XLenVT), MGN->getMask(),
EVL, MGN->getMemOperand());
- SDValue VPSelect = DAG.getNode(ISD::VP_SELECT, DL, VT, MGN->getMask(),
- StridedLoad, MGN->getPassThru(), EVL);
- return DAG.getMergeValues({VPSelect, SDValue(StridedLoad.getNode(), 1)},
+ SDValue Select = DAG.getSelect(DL, VT, MGN->getMask(), StridedLoad,
+ MGN->getPassThru());
+ return DAG.getMergeValues({Select, SDValue(StridedLoad.getNode(), 1)},
DL);
}
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/157070
More information about the llvm-commits
mailing list