[PATCH] D43288: [AArch64] Add support for secrel add/load/store relocations for COFF
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 1 06:19:15 PST 2018
mstorsjo added a comment.
@compnerd wanted some more opinions on this from @rnk and @majnemer. He also wanted to make sure that the new relocations are forbidden if targeting ELF; this already happens, e.g. like this:
test.s:2:2: error: invalid fixup for 64-bit load/store instruction
ldr x0, [x0, :secrel_lo12:foo]
These aren't strictly necessary to expose on the assembler level in order to implement the next step (actually supporting native TLS), but having them supported in the assembler is useful for testing and for completeness sake.
More information about the llvm-commits