[PATCH] D139687: [AVR] Select 16-bit LDS/STS for load/store on AVRTiny.

Ayke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 22 07:38:36 PST 2022


aykevl accepted this revision.
aykevl added a comment.
This revision is now accepted and ready to land.

Looks good to me



================
Comment at: llvm/lib/Target/AVR/AVRInstrInfo.td:216
 // A 7-bit address (which can lead to an R_AVR_LDS_STS_16 relocation).
-def imm7tiny : Operand<i8> {
+def imm7tiny : Operand<i16> {
   let EncoderMethod = "encodeImm<AVR::fixup_lds_sts_16, 0>";
----------------
It would have been a bit cleaner to include this in the previous patch but it's fine now.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139687/new/

https://reviews.llvm.org/D139687



More information about the llvm-commits mailing list