[llvm] e5cd716 - [PowerPC] Remove assumptions about subreg index values. NFC. (#219524)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 28 13:03:21 PDT 2026
Author: Jay Foad
Date: 2026-08-28T21:03:17+01:00
New Revision: e5cd716940554b46bcaef9419a801786d409207a
URL: https://github.com/llvm/llvm-project/commit/e5cd716940554b46bcaef9419a801786d409207a
DIFF: https://github.com/llvm/llvm-project/commit/e5cd716940554b46bcaef9419a801786d409207a.diff
LOG: [PowerPC] Remove assumptions about subreg index values. NFC. (#219524)
Use subreg index names instead of hard coded values in a few places in
the backend.
Added:
Modified:
llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp
llvm/lib/Target/PowerPC/PPCISelLowering.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp b/llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp
index 016c547821f7a..6131657ba1b8f 100644
--- a/llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp
+++ b/llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp
@@ -3223,7 +3223,7 @@ SDValue IntegerCompareEliminator::addExtOrTrunc(SDValue NatWidthRes,
SDLoc dl(NatWidthRes);
// For reinterpreting 32-bit values as 64 bit values, we generate
- // INSERT_SUBREG IMPLICIT_DEF:i64, <input>, TargetConstant:i32<1>
+ // INSERT_SUBREG IMPLICIT_DEF:i64, <input>, TargetConstant:i32<%subreg.sub_32>
if (Conv == ExtOrTruncConversion::Ext) {
SDValue ImDef(CurDAG->getMachineNode(PPC::IMPLICIT_DEF, dl, MVT::i64), 0);
SDValue SubRegIdx =
@@ -5183,7 +5183,7 @@ bool PPCDAGToDAGISel::tryAsSingleRLDICL(SDNode *N) {
Val = SDValue(CurDAG->getMachineNode(PPC::INSERT_SUBREG, dl, ResultType,
IDVal, Op0.getOperand(0),
- getI32Imm(1, dl)),
+ getI32Imm(PPC::sub_32, dl)),
0);
SH = 64 - Imm;
}
diff --git a/llvm/lib/Target/PowerPC/PPCISelLowering.cpp b/llvm/lib/Target/PowerPC/PPCISelLowering.cpp
index d599bba54079b..7c94f5163845d 100644
--- a/llvm/lib/Target/PowerPC/PPCISelLowering.cpp
+++ b/llvm/lib/Target/PowerPC/PPCISelLowering.cpp
@@ -14922,9 +14922,9 @@ PPCTargetLowering::EmitInstrWithCustomInserter(MachineInstr &MI,
// IMPLICIT_DEF register.
BuildMI(*BB, MI, dl, TII->get(TargetOpcode::IMPLICIT_DEF), ImDefReg);
BuildMI(*BB, MI, dl, TII->get(PPC::INSERT_SUBREG), ExtSrcReg)
- .addReg(ImDefReg)
- .add(SrcOp)
- .addImm(1);
+ .addReg(ImDefReg)
+ .add(SrcOp)
+ .addImm(PPC::sub_32);
Register NewFPSCRTmpReg = RegInfo.createVirtualRegister(&PPC::G8RCRegClass);
BuildMI(*BB, MI, dl, TII->get(PPC::RLDIMI), NewFPSCRTmpReg)
More information about the llvm-commits
mailing list