This is an archive of the discontinued LLVM Phabricator instance.

[AARCH64] Incorrect relocation name TLSDESC_LD64_LO12_NC
AbandonedPublic

Authored by joelkevinjones on Feb 28 2017, 2:22 PM.

Details

Summary

The document "ELF for the ARM 64-bit Architecture (AArch64)" at http://infocenter.arm.com/help/topic/com.arm.doc.ihi0056c/IHI0056C_beta_aaelf64.pdf
gives the name as R_AARCH64_TLSDESC_LD64_LO12 (NB: without the trailing "_NC").

This patch fixes the naming, adjusts tests, and modifies the asm printer to remove the VK_NC target flag when expanding TLSDESC_CALLSEQ.

Diff Detail

Repository
rL LLVM

Event Timeline

joelkevinjones created this revision.Feb 28 2017, 2:22 PM
joelkevinjones abandoned this revision.Apr 13 2017, 9:05 PM

This revision is included in an omnibus ILP32 relocation change: https://reviews.llvm.org/D32072