[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 mailing list