[llvm] [LV] Vectorize selecting last IV of min/max element. (PR #141431)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 19 07:11:05 PST 2025
================
@@ -214,6 +214,40 @@ static bool checkOrderedReduction(RecurKind Kind, Instruction *ExactFPMathInst,
return true;
}
+static std::optional<RecurrenceDescriptor>
+getMultiUseMinMax(PHINode *Phi, RecurKind Kind, Loop *TheLoop) {
+ BasicBlock *Latch = TheLoop->getLoopLatch();
+ if (!Latch)
+ return std::nullopt;
+ Value *Inc = Phi->getIncomingValueForBlock(Latch);
+ RecurKind RK;
+ if (Phi->hasOneUse() ||
+ !RecurrenceDescriptor::isIntMinMaxRecurrenceKind(Kind))
+ return std::nullopt;
+
+ Value *A, *B;
+ if (match(Inc, m_OneUse(m_UMin(m_Value(A), m_Value(B)))))
+ RK = RecurKind::UMin;
+ else if (match(Inc, m_OneUse(m_UMax(m_Value(A), m_Value(B)))))
+ RK = RecurKind::UMax;
+ else if (match(Inc, m_OneUse(m_SMax(m_Value(A), m_Value(B)))))
+ RK = RecurKind::SMax;
+ else if (match(Inc, m_OneUse(m_SMin(m_Value(A), m_Value(B)))))
+ RK = RecurKind::SMin;
+ else
+ return std::nullopt;
----------------
fhahn wrote:
Above checks the requested RecurrenceKind, here we check the incoming value, but the code was a bit confusing and didn't account properly for the requested Kind. Updated to remove setting RK, and only use the appropriate matcher for `Kind`, thanks
https://github.com/llvm/llvm-project/pull/141431
More information about the llvm-commits
mailing list