[llvm] [RISCV] Refactor performCONCAT_VECTORSCombine. NFC (PR #69068)
Michael Maitland via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 16 08:37:05 PDT 2023
================
@@ -13867,26 +13844,25 @@ static SDValue performCONCAT_VECTORSCombine(SDNode *N, SelectionDAG &DAG,
if (!TLI.isLegalStridedLoadStore(WideVecVT, Align))
return SDValue();
+ auto [Stride, Reversed] = *BaseDiff;
----------------
michaelmaitland wrote:
Looking at #68726, I wonder if `Reversed` may not be the best name for this variable? The RVV spec uses the term `negative stride`. Additionally, when `BaseIndexOffset` returns a negative value, it does not need to be negated here although that load is "in reverse". I suggest we rename this to something along the lines of `MustNegateStride`?
https://github.com/llvm/llvm-project/pull/69068
More information about the llvm-commits
mailing list