[PATCH] D74213: [AVR] Don't adjust addresses by 2 for absolute values

Ayke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 7 13:30:36 PST 2020


aykevl added a comment.

After some investigation, it appears that "temporary labels" can be constructed like this:

  1:
      jmp 1b

I tried to make a test for this but it seems like `adjustFixupValue` is only called when actually generating an object file. Therefore, to test this we'd need to disassemble the code and check whether the raw bytes have the correct relocation addend. So I think a good test for this would need to wait until there is better disassembly support.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74213





More information about the llvm-commits mailing list