[PATCH] D65857: [MC][AArch64] Restrict use of signed relocation operators on MOV[NZK]

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 16 15:47:45 PST 2023


pcc added a comment.
Herald added a project: All.

In D65857#1621377 <https://reviews.llvm.org/D65857#1621377>, @peter.smith wrote:

> Thanks for getting back to me, I'll look into making a strict mode. I'll also mention the use case you've identified with movk to my colleague from our GCC team, it is a pity that we don't have a R_AARCH64_PREL_G3_NC for the movk as although not ideal, as it is skipping an overflow check, it would permit GNU ld to be extended whilst retaining the property that you don't need to disassemble the binary to know how to evaluate a relocation.

Did you ever get a chance to mention this to the GCC team? I've heard that binutils is still relocating MOVK into an illegal instruction.


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

https://reviews.llvm.org/D65857



More information about the llvm-commits mailing list