[PATCH] D64466: AArch64: Unify relocation restrictions between MOVK/MOVN/MOVZ.

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 15 06:53:04 PDT 2019


peter.smith added a comment.

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

> I think that this is reasonable given that the wording in the ABI was recently changed to relax some of the wording surrounding MOVZ and MOVK. In particular Table 4-7 and Table 4-8 where modified in version E, the latest version is F https://developer.arm.com/docs/ihi0056/latest/elf-for-the-arm-64-bit-architecture-aarch64-abi-2019q2-documentation
>
> Unfortunately it looks like Table 4-11 and Table 4-12 still have the distinction between MOVZ and MOVK. I think that this is likely an oversight rather than a deliberate decision, but I'd like to check. I'm on holiday this week, so I'll need to wait till Monday to follow this up. I've added a couple of my colleagues who may be able to help in my absence.


I've submitted a query to the ABI mailing list to see if anyone has a particular reason for not doing this for the pc and got relative versions. If no-one comes back with any good reason within a couple of days I'll approve.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64466





More information about the llvm-commits mailing list