[PATCH] D98946: [AIX][TLS] Add support for TLS variables to XCOFF object writer

[Hubert Tong via Phabricator via llvm-commits]

hubert.reinterpretcast added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/aix-tls-xcoff-variables.ll:37-38
+; SECTION-NEXT:     Name: .tdata
+; SECTION-DATASECT:        PhysicalAddress: 0x4
+; SECTION-DATASECT-NEXT:   VirtualAddress: 0x4
+; SECTION-DATASECT-NEXT:   Size: 0x2C
----------------
This doesn't match the behaviour of the AIX assembler (and also of the XL compiler). The `.tdata` section starts again at address `0x0`.

It seems the documentation is wrong one way or the other though. It says (regarding `s_paddr`):
> This field should contain 0 for all sections except the .text , .data , and .bss sections.

but it is not the case that the `.tbss` starts again at `0x0` (it does follow `.tdata`).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98946