[llvm] [SPARC] Emit a TRUNCATE when combining truncating BSWAP-STOREs (PR #210483)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 31 10:16:31 PDT 2026
================
@@ -3333,6 +3333,9 @@ SDValue SparcTargetLowering::PerformSTORECombine(SDNode *N,
unsigned Shift = VT.getSizeInBits() - MemVT.getSizeInBits();
BSwapOp = DAG.getNode(ISD::SRL, DL, VT, BSwapOp,
DAG.getShiftAmountConstant(Shift, VT, DL));
+ // Need to truncate if this is a bswap of i64 stored as i32/i16.
+ if (VT == MVT::i64)
+ BSwapOp = DAG.getNode(ISD::TRUNCATE, DL, MVT::i32, BSwapOp);
----------------
koachan wrote:
> > I'm thinking that the extra patterns would match the already defined i32->i16 one...
>
> I'm not sure I undersnant what you're trying to say?..
Was just thinking that I already defined i32->i16 patterns anyway so defining additional patterns for i64->i32 and i64->i16 will be cleaner than special casing it through TRUNCATEs.
https://github.com/llvm/llvm-project/pull/210483
More information about the llvm-commits
mailing list