[llvm] Promote 32bit pseudo instr that infer extsw removal to 64bit in PPCMIPeephole (PR #85451)
Lei Huang via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 23 12:41:03 PDT 2024
================
@@ -5234,6 +5234,218 @@ bool PPCInstrInfo::isTOCSaveMI(const MachineInstr &MI) const {
// We limit the max depth to track incoming values of PHIs or binary ops
// (e.g. AND) to avoid excessive cost.
const unsigned MAX_BINOP_DEPTH = 1;
+
+// This function will promote the instruction which defines the register `Reg`
+// in the parameter from a 32-bit to a 64-bit instruction if needed. The logic
+// used to check whether an instruction needs to be promoted or not is similar
+// to the logic used to check whether or not a defined register is sign or zero
+// extended within the function PPCInstrInfo::isSignOrZeroExtended.
+// Additionally, the `promoteInstr32To64ForElimEXTSW` function is recursive.
+// BinOpDepth does not count all of the recursions. The parameter BinOpDepth is
+// incremented only when `promoteInstr32To64ForElimEXTSW` calls itself more
+// than once. This is done to prevent exponential recursion.
+void PPCInstrInfo::promoteInstr32To64ForElimEXTSW(const Register &Reg,
+ MachineRegisterInfo *MRI,
+ unsigned BinOpDepth,
+ LiveVariables *LV) const {
+ if (!Reg.isVirtual())
+ return;
+
+ MachineInstr *MI = MRI->getVRegDef(Reg);
+ if (!MI)
+ return;
+
+ unsigned Opcode = MI->getOpcode();
+
+ switch (Opcode) {
+ case PPC::OR:
+ case PPC::ISEL:
+ case PPC::OR8:
+ case PPC::PHI: {
+ if (BinOpDepth >= MAX_BINOP_DEPTH)
+ break;
+ unsigned OperandEnd = 3, OperandStride = 1;
+ if (Opcode == PPC::PHI) {
+ OperandEnd = MI->getNumOperands();
+ OperandStride = 2;
+ }
+
+ for (unsigned I = 1; I < OperandEnd; I += OperandStride) {
+ assert(MI->getOperand(I).isReg() && "Operand must be register");
+ promoteInstr32To64ForElimEXTSW(MI->getOperand(I).getReg(), MRI,
+ BinOpDepth + 1, LV);
+ }
+
+ break;
+ }
+ case PPC::COPY: {
+ // Refers to the logic of the `case PPC::COPY` statement in the function
+ // PPCInstrInfo::isSignOrZeroExtended().
+
+ Register SrcReg = MI->getOperand(1).getReg();
+ // In both ELFv1 and v2 ABI, method parameters and the return value
+ // are sign- or zero-extended.
+ const MachineFunction *MF = MI->getMF();
+ if (!MF->getSubtarget<PPCSubtarget>().isSVR4ABI()) {
+ // If this is a copy from another register, we recursively promote the
+ // source.
+ promoteInstr32To64ForElimEXTSW(SrcReg, MRI, BinOpDepth, LV);
+ return;
+ }
+
+ // From here on everything is SVR4ABI. COPY will be eliminated in the other
+ // pass, we do not need promote the COPY pseudo opcode.
+
+ if (SrcReg != PPC::X3)
+ // If this is a copy from another register, we recursively promote the
+ // source.
+ promoteInstr32To64ForElimEXTSW(SrcReg, MRI, BinOpDepth, LV);
+ return;
+ }
+ case PPC::ORI:
+ case PPC::XORI:
+ case PPC::ORIS:
+ case PPC::XORIS:
+ case PPC::ORI8:
+ case PPC::XORI8:
+ case PPC::ORIS8:
+ case PPC::XORIS8:
+ promoteInstr32To64ForElimEXTSW(MI->getOperand(1).getReg(), MRI, BinOpDepth,
+ LV);
+ break;
+ case PPC::AND:
+ case PPC::AND8:
+ if (BinOpDepth >= MAX_BINOP_DEPTH)
+ break;
+
+ promoteInstr32To64ForElimEXTSW(MI->getOperand(1).getReg(), MRI,
+ BinOpDepth + 1, LV);
+ promoteInstr32To64ForElimEXTSW(MI->getOperand(2).getReg(), MRI,
+ BinOpDepth + 1, LV);
+ break;
+ }
+
+ bool HasNonSignedExtInstrPromoted = false;
+ int NewOpcode = -1;
+
+ // Map the opcode of instructions (which are not sign- or zero-extended
+ // themselves,but have operands that are destination registers of sign- or
+ // zero-extended instructions) to their 64-bit equivalents.
----------------
lei137 wrote:
```suggestion
// Map the 32bit to 64bit opcodes for instructions that are not signed or zero extended
// themselves, but may have operands who's destination registers are of signed or
// zero extended instructions.
```
https://github.com/llvm/llvm-project/pull/85451
More information about the llvm-commits
mailing list