[PATCH] D146216: [lld][ELF] Relax long jump/call to short jump/call on AVR

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: lld/ELF/Arch/AVR.cpp:245
+//     rjmp _foo  ; 2-byte instruction
+//     nop        ; 2-byte instruction
+bool AVR::tryRelaxLongJumpCall(uint8_t *loc, uint64_t callAddr,
----------------
benshi001 wrote:
> Is there any better we can remove this nop ? I can only figure out the way by `memcpy` `uint8_t *loc` .
I implemented this for RISC-V but I am unsure we should get the complexity for the less-used AVR port. This complexity is exactly what I called out in a previous patch and you said that you did not intend to implement it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146216