combine 34bit print with 32bit

Author: lei137

llvm/lib/Target/PowerPC/MCTargetDesc/PPCInstPrinter.cpp

@@ -430,29 +430,18 @@ void PPCInstPrinter::printS16ImmOperand(const MCInst *MI, unsigned OpNo,
     printOperand(MI, OpNo, STI, O);
 }
 
-void PPCInstPrinter::printS32ImmOperand(const MCInst *MI, unsigned OpNo,
+template<uint64_t N>
+void PPCInstPrinter::printSImmOperand(const MCInst *MI, unsigned OpNo,
                                         const MCSubtargetInfo &STI,
                                         raw_ostream &O) {
   if (MI->getOperand(OpNo).isImm()) {
-    long long Value = MI->getOperand(OpNo).getImm();
-    assert(isInt<32>(Value) && "Invalid s32imm argument!");
-    O << (long long)Value;
+    int64_t Value = MI->getOperand(OpNo).getImm();
+    assert(isInt<N>(Value) && "Invalid imm argument!");
+    O << (int64_t)Value;
   } else
     printOperand(MI, OpNo, STI, O);
 }
 
-void PPCInstPrinter::printS34ImmOperand(const MCInst *MI, unsigned OpNo,
-                                        const MCSubtargetInfo &STI,
-                                        raw_ostream &O) {
-  if (MI->getOperand(OpNo).isImm()) {
-    long long Value = MI->getOperand(OpNo).getImm();
-    assert(isInt<34>(Value) && "Invalid s34imm argument!");
-    O << (long long)Value;
-  }
-  else
-    printOperand(MI, OpNo, STI, O);
-}
-
 void PPCInstPrinter::printU16ImmOperand(const MCInst *MI, unsigned OpNo,
                                         const MCSubtargetInfo &STI,
                                         raw_ostream &O) {
@@ -543,7 +532,7 @@ void PPCInstPrinter::printMemRegImmHash(const MCInst *MI, unsigned OpNo,
 void PPCInstPrinter::printMemRegImm34PCRel(const MCInst *MI, unsigned OpNo,
                                            const MCSubtargetInfo &STI,
                                            raw_ostream &O) {
-  printS34ImmOperand(MI, OpNo, STI, O);
+  printSImmOperand<34>(MI, OpNo, STI, O);
   O << '(';
   printImmZeroOperand(MI, OpNo + 1, STI, O);
   O << ')';
@@ -552,7 +541,7 @@ void PPCInstPrinter::printMemRegImm34PCRel(const MCInst *MI, unsigned OpNo,
 void PPCInstPrinter::printMemRegImm34(const MCInst *MI, unsigned OpNo,
                                       const MCSubtargetInfo &STI,
                                       raw_ostream &O) {
-  printS34ImmOperand(MI, OpNo, STI, O);
+  printSImmOperand<34>(MI, OpNo, STI, O);
   O << '(';
   printOperand(MI, OpNo + 1, STI, O);
   O << ')';

llvm/lib/Target/PowerPC/MCTargetDesc/PPCInstPrinter.h

@@ -80,9 +80,8 @@ class PPCInstPrinter : public MCInstPrinter {
                           const MCSubtargetInfo &STI, raw_ostream &O);
   void printS16ImmOperand(const MCInst *MI, unsigned OpNo,
                           const MCSubtargetInfo &STI, raw_ostream &O);
-  void printS32ImmOperand(const MCInst *MI, unsigned OpNo,
-                          const MCSubtargetInfo &STI, raw_ostream &O);
-  void printS34ImmOperand(const MCInst *MI, unsigned OpNo,
+  template<uint64_t N>
+  void printSImmOperand(const MCInst *MI, unsigned OpNo,
                           const MCSubtargetInfo &STI, raw_ostream &O);
   void printU16ImmOperand(const MCInst *MI, unsigned OpNo,
                           const MCSubtargetInfo &STI, raw_ostream &O);

llvm/lib/Target/PowerPC/PPCRegisterInfo.td

@@ -814,14 +814,14 @@ def PPCS32ImmAsmOperand : AsmOperandClass {
   let RenderMethod = "addImmOperands";
 }
 def s32imm : Operand<i64> {
-  let PrintMethod = "printS32ImmOperand";
+  let PrintMethod = "printSImmOperand<32>";
   let EncoderMethod = "getImmEncoding<PPC::fixup_ppc_imm32>";
   let ParserMatchClass = PPCS32ImmAsmOperand;
   let DecoderMethod = "decodeSImmOperand<32>";
   let OperandType = "OPERAND_IMMEDIATE";
 }
 def s32imm_pcrel : Operand<i64> {
-  let PrintMethod = "printS32ImmOperand";
+  let PrintMethod = "printSImmOperand<32>";
   let EncoderMethod = "getImmEncoding<PPC::fixup_ppc_pcrel32>";
   let ParserMatchClass = PPCS32ImmAsmOperand;
   let DecoderMethod = "decodeSImmOperand<32>";
@@ -833,14 +833,14 @@ def PPCS34ImmAsmOperand : AsmOperandClass {
   let RenderMethod = "addImmOperands";
 }
 def s34imm : Operand<i64> {
-  let PrintMethod = "printS34ImmOperand";
+  let PrintMethod = "printSImmOperand<34>";
   let EncoderMethod = "getImmEncoding<PPC::fixup_ppc_imm34>";
   let ParserMatchClass = PPCS34ImmAsmOperand;
   let DecoderMethod = "decodeSImmOperand<34>";
   let OperandType = "OPERAND_IMMEDIATE";
 }
 def s34imm_pcrel : Operand<i64> {
-  let PrintMethod = "printS34ImmOperand";
+  let PrintMethod = "printSImmOperand<34>";
   let EncoderMethod = "getImmEncoding<PPC::fixup_ppc_pcrel34>";
   let ParserMatchClass = PPCS34ImmAsmOperand;
   let DecoderMethod = "decodeSImmOperand<34>";