[AVR] Add PrintMethod for operand memspi

[benshi001]

It is hard to make a test for my change, since operand memspi is used only by the STDSPQRr and STDWSPQRr pseudo instruction. Adding PrintMethod is just for debug purpose, and we should see no pseudo instruction in the final assembly file.

[benshi001]

@Patryk27 Could you please have a look at this change? I find this bug when debugging your https://reviews.llvm.org/D123528. The STDSPQRr & STDWSPQRr are printed without the offset vaule.

[Patryk27]

The TableGen stuff is still dark magic to me, but the change looks easy & reasonable :-)

[benshi001]

It is hard to make a test for my change, since operand memspi is used only by the STDSPQRr and STDWSPQRr pseudo instructions. Adding PrintMethod is just for debug purpose, and we should see no pseudo instruction in the final assembly file.

llvm/lib/Target/AVR/AVRInstrInfo.td

def memri : Operand<iPTR> {
   let PrintMethod = "printMemri";
   let EncoderMethod = "encodeMemri";
   let DecoderMethod = "decodeMemri";
 
   let ParserMatchClass = MemriAsmOperand;
 }
 
 // Address operand for `SP+imm` used by STD{W}SPQRr
-def memspi : Operand<iPTR> { let MIOperandInfo = (ops GPRSP, i16imm); }
+def memspi : Operand<iPTR> {
+  let MIOperandInfo = (ops GPRSP, i16imm);
+  let PrintMethod = "printMemspi";
+}
 
 def relbrtarget_7 : Operand<OtherVT> {
   let PrintMethod = "printPCRelImm";
   let EncoderMethod = "encodeRelCondBrTarget<AVR::fixup_7_pcrel>";
 }
 
 def brtarget_13 : Operand<OtherVT> {
   let PrintMethod = "printPCRelImm";

llvm/lib/Target/AVR/MCTargetDesc/AVRInstPrinter.h

private:
 
   void printOperand(const MCInst *MI, unsigned OpNo, raw_ostream &O);
   void printPCRelImm(const MCInst *MI, unsigned OpNo, raw_ostream &O);
   void printPCRelImm(const MCInst *MI, uint64_t /*Address*/, unsigned OpNo,
                      raw_ostream &O) {
     printPCRelImm(MI, OpNo, O);
   }
   void printMemri(const MCInst *MI, unsigned OpNo, raw_ostream &O);
+  void printMemspi(const MCInst *MI, unsigned OpNo, raw_ostream &O) {
+    printMemri(MI, OpNo, O);
+  }
 
   // Autogenerated by TableGen.
   std::pair<const char *, uint64_t> getMnemonic(const MCInst *MI) override;
   void printInstruction(const MCInst *MI, uint64_t Address, raw_ostream &O);
   bool printAliasInstr(const MCInst *MI, uint64_t Address, raw_ostream &O);
   void printCustomAliasOperand(const MCInst *MI, uint64_t Address,
                                unsigned OpIdx, unsigned PrintMethodIdx,
                                raw_ostream &O);
 };
 
 } // end namespace llvm
 
 #endif // LLVM_AVR_INST_PRINTER_H