[llvm] [DAG][X86] Use pattern matching to simplify PSADBW+ADD combine (PR #147637)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 8 19:18:53 PDT 2025
https://github.com/woruyu created https://github.com/llvm/llvm-project/pull/147637
### Summary
This patch refactors the add(psadbw(x, 0), psadbw(y, 0)) -> psadbw(x + y, 0) combine to use SelectionDAG::sd_match and SDPatternMatch matchers instead of manually checking opcodes and operands.
>From b0a70d14cfdba3d3b2ea8511770c0d1240d97020 Mon Sep 17 00:00:00 2001
From: woruyu <1214539920 at qq.com>
Date: Wed, 9 Jul 2025 10:14:24 +0800
Subject: [PATCH] feat: use pattern matching to simplify PSADBW+ADD combine
---
llvm/lib/Target/X86/X86ISelLowering.cpp | 20 +++++++++-----------
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp
index fd617f7062313..9baa51ea36a86 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -57867,22 +57867,20 @@ static SDValue combineAdd(SDNode *N, SelectionDAG &DAG,
}
}
+ SDValue X, Y;
+
// add(psadbw(X,0),psadbw(Y,0)) -> psadbw(add(X,Y),0)
// iff X and Y won't overflow.
- if (Op0.getOpcode() == X86ISD::PSADBW && Op1.getOpcode() == X86ISD::PSADBW &&
- ISD::isBuildVectorAllZeros(Op0.getOperand(1).getNode()) &&
- ISD::isBuildVectorAllZeros(Op1.getOperand(1).getNode())) {
- if (DAG.willNotOverflowAdd(false, Op0.getOperand(0), Op1.getOperand(0))) {
- MVT OpVT = Op0.getOperand(1).getSimpleValueType();
- SDValue Sum =
- DAG.getNode(ISD::ADD, DL, OpVT, Op0.getOperand(0), Op1.getOperand(0));
- return DAG.getNode(X86ISD::PSADBW, DL, VT, Sum,
- getZeroVector(OpVT, Subtarget, DAG, DL));
- }
+ if (sd_match(Op0, m_c_BinOp(X86ISD::PSADBW, m_Value(X), m_Zero())) &&
+ sd_match(Op1, m_c_BinOp(X86ISD::PSADBW, m_Value(Y), m_Zero())) &&
+ DAG.willNotOverflowAdd(/*IsSigned=*/false, X, Y)) {
+ MVT OpVT = X.getSimpleValueType();
+ SDValue Sum = DAG.getNode(ISD::ADD, DL, OpVT, X, Y);
+ return DAG.getNode(X86ISD::PSADBW, DL, VT, Sum,
+ getZeroVector(OpVT, Subtarget, DAG, DL));
}
if (VT.isVector()) {
- SDValue X, Y;
EVT BoolVT = EVT::getVectorVT(*DAG.getContext(), MVT::i1,
VT.getVectorElementCount());
More information about the llvm-commits
mailing list