[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