[AVRInstPrinter] printOperand: support llvm-objdump --print-imm-hex

[MaskRay]

Forgot to add test

[aykevl]

Looks good to me. I've confirmed this works with D78741 and I have updated the patch to make use of it.

[MaskRay]

In D83634#2146236, @aykevl wrote:

Looks good to me. I've confirmed this works with D78741 and I have updated the patch to make use of it.

Thanks. I'll push this to unblock D78741.

[aykevl]

Thanks. I'll push this to unblock D78741.

Awesome, thank you! I merged that change as well so LLVM 11 should get AVR lld support.

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

bool isPtrReg = (MOI.RegClass == AVR::PTRREGSRegClassID) ||
                     (MOI.RegClass == AVR::ZREGRegClassID);
 
     if (isPtrReg) {
       O << getRegisterName(Op.getReg(), AVR::ptr);
     } else {
       O << getPrettyRegisterName(Op.getReg(), MRI);
     }
   } else if (Op.isImm()) {
-    O << Op.getImm();
+    O << formatImm(Op.getImm());
   } else {
     assert(Op.isExpr() && "Unknown operand kind in printOperand");
     O << *Op.getExpr();
   }
 }
 
 /// This is used to print an immediate value that ends up
 /// being encoded as a pc-relative value.

llvm/test/MC/AVR/hex-immediates.s

+; RUN: llvm-mc -filetype=obj -triple=avr %s -o %t
+; RUN: llvm-objdump -d %t | FileCheck %s --check-prefix=DEC
+; RUN: llvm-objdump -d --print-imm-hex %t | FileCheck %s --check-prefix=HEX
+
+; DEC: ldi r24, 66
+; HEX: ldi r24, 0x42
+  ldi r24, 0x42