[PATCH] D27308: ELF/AArch64: Fix R_AARCH64_LDST16_ABS_LO12_NC mask

Adhemerval Zanella via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 2 12:34:54 PST 2016


zatrazz added a comment.

In https://reviews.llvm.org/D27308#612006, @meadori wrote:

> LGTM.  It is constant with my understanding of: http://infocenter.arm.com/help/topic/com.arm.doc.ihi0056b/IHI0056B_aaelf64.pdf
>
> In looking at the surrounding similar relocations, shouldn't `R_AARCH64_LDST128_ABS_LO12_NC` be `0xFF0`?


Yes, this is on my list. I am struggling to create a testcase that triggers an issue with correct invalid mask.


Repository:
  rL LLVM

https://reviews.llvm.org/D27308





More information about the llvm-commits mailing list