[llvm] 90961df - [VPlan] Strip phi operand from compute-reduction-result comments (NFC).
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 14 11:41:14 PST 2026
Author: Florian Hahn
Date: 2026-01-14T19:39:43Z
New Revision: 90961df83b998aab75de4e0410b712a598b0c5e7
URL: https://github.com/llvm/llvm-project/commit/90961df83b998aab75de4e0410b712a598b0c5e7
DIFF: https://github.com/llvm/llvm-project/commit/90961df83b998aab75de4e0410b712a598b0c5e7.diff
LOG: [VPlan] Strip phi operand from compute-reduction-result comments (NFC).
After d5c11b9a24c84f1, compute-reduction-result does not have the
reduction phi recipe as operand. Update stale comments pointed out
independently in https://github.com/llvm/llvm-project/pull/175461.
Added:
Modified:
llvm/lib/Transforms/Vectorize/VPlanConstruction.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Transforms/Vectorize/VPlanConstruction.cpp b/llvm/lib/Transforms/Vectorize/VPlanConstruction.cpp
index e60d0f4c31bd1..bf8b0461e3b1b 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanConstruction.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlanConstruction.cpp
@@ -1491,14 +1491,13 @@ bool VPlanTransforms::handleMultiUseReductions(VPlan &Plan) {
// correspond to the lanes matching the min/max reduction result.
//
// For example, this transforms
- // vp<%min.result> = compute-reduction-result ir<%min.val>,
- // ir<%min.val.next>
+ // vp<%min.result> = compute-reduction-result ir<%min.val.next>
// vp<%find.iv.result = compute-find-iv-result ir<%min.idx>, ir<0>,
// SENTINEL, vp<%min.idx.next>
//
// into:
//
- // vp<min.result> = compute-reduction-result ir<%min.val>, ir<%min.val.next>
+ // vp<min.result> = compute-reduction-result ir<%min.val.next>
// vp<%final.min.cmp> = icmp eq ir<%min.val.next>, vp<min.result>
// vp<%final.iv> = select vp<%final.min.cmp>, ir<%min.idx.next>, SENTINEL
// vp<%find.iv.result> = compute-find-iv-result ir<%min.idx>, ir<0>,
More information about the llvm-commits
mailing list