[PATCH] D113509: [lld][ELF] Support for R_ARM_THM_JUMP8

[Peter Smith via Phabricator via llvm-commits]

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

Thanks for the updates LGTM. In theory we can now rewrite arm-thumb-narrow-branch-check.s using .reloc but that could be done in a future patch.



================
Comment at: lld/test/ELF/Inputs/arm-thumb-narrow-branch.s:4
+// branch. This is permissible by the ABI for the ARM architecture as the range
+// of the Thumb narrow branch is short enough (+- 2048 bytes) that widespread
+// use would be impractical.
----------------
R_ARM_THM_JUMP8 has an even smaller range of +- 256 bytes, but I don't think it is worth changing the comment as the point still stands.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113509