[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