[PATCH] D100214: [AIX][TLS] Add support for TLSGD relocations to XCOFF objects

[Nemanja Ivanovic via Phabricator via llvm-commits]

nemanjai added a comment.

In D100214#2701875 <https://reviews.llvm.org/D100214#2701875>, @sfertile wrote:

> Patch looks pretty good. IIUC we were not printing the '@gd' specifier before because it is assumed by default by the system assembler. Now that we are adding object file support we can no longer rely of the system assembler to assume the specifier based on the storage mapping class ,and need to add the new variant kinds so we create the correct relocations, which has the side effect of also changing the asm we emit. I know I am being pedantic, but I think we should land that part of the change and the update to the asm test in an initial patch even though the object writing is the motivation for adding it.

I agree, lets split out the ASM portion from the object code portion.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100214/new/

https://reviews.llvm.org/D100214