[PATCH] D43288: [AArch64] Add support for secrel add/load/store relocations for COFF

[Martin Storsjö via Phabricator via llvm-commits]

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.


https://reviews.llvm.org/D43288