[llvm] [X86] combineINSERT_SUBVECTOR - use getBROADCAST_LOAD helper in insert_subvector(undef, broadcast(p), hi) -> broadcast(p) fold (PR #128900)
Simon Pilgrim via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 26 08:12:56 PST 2025
https://github.com/RKSimon created https://github.com/llvm/llvm-project/pull/128900
None
>From 86e52d671140278eba934776f19a8e395c83ae19 Mon Sep 17 00:00:00 2001
From: Simon Pilgrim <llvm-dev at redking.me.uk>
Date: Wed, 26 Feb 2025 16:12:11 +0000
Subject: [PATCH] [X86] combineINSERT_SUBVECTOR - use getBROADCAST_LOAD helper
in insert_subvector(undef, broadcast(p), hi) -> broadcast(p) fold
---
llvm/lib/Target/X86/X86ISelLowering.cpp | 10 ++--------
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp
index 9a259fef719f5..0dc73bbf8e3f9 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -58555,14 +58555,8 @@ static SDValue combineINSERT_SUBVECTOR(SDNode *N, SelectionDAG &DAG,
if (Vec.isUndef() && IdxVal != 0 && SubVec.hasOneUse() &&
SubVec.getOpcode() == X86ISD::VBROADCAST_LOAD) {
auto *MemIntr = cast<MemIntrinsicSDNode>(SubVec);
- SDVTList Tys = DAG.getVTList(OpVT, MVT::Other);
- SDValue Ops[] = { MemIntr->getChain(), MemIntr->getBasePtr() };
- SDValue BcastLd =
- DAG.getMemIntrinsicNode(X86ISD::VBROADCAST_LOAD, dl, Tys, Ops,
- MemIntr->getMemoryVT(),
- MemIntr->getMemOperand());
- DAG.ReplaceAllUsesOfValueWith(SDValue(MemIntr, 1), BcastLd.getValue(1));
- return BcastLd;
+ return getBROADCAST_LOAD(X86ISD::VBROADCAST_LOAD, dl, OpVT,
+ MemIntr->getMemoryVT(), MemIntr, 0, DAG);
}
// If we're splatting the lower half subvector of a full vector load into the
More information about the llvm-commits
mailing list