[PATCH] D149722: [AIX][TLS] Generate 64-bit local-exec access code sequence

Digger Lin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 11:52:08 PDT 2023


DiggerLin added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/aix-tls-le-xcoff-reloc-large.ll:150
+; SYM-NEXT:     Index: 0
+; SYM-NEXT:     Name: <stdin>
+; SYM-NEXT:     Value (SymbolTableIndex): 0x0
----------------
in order to make the test case more readable, suggest to remove all not related the local_exec TLS function CHECK.  


================
Comment at: llvm/test/CodeGen/PowerPC/aix-tls-le-xcoff-reloc-large.ll:274
+; SYM-NEXT:     Index: 13
+; SYM-NEXT:     Name: TOC
+; SYM-NEXT:     Value (RelocatableAddress): 0x80
----------------
ditto


================
Comment at: llvm/test/CodeGen/PowerPC/aix-tls-le-xcoff-reloc.ll:176
+; SYM-NEXT:     Value (RelocatableAddress): 0x10
+; SYM-NEXT:     Section: .text
+; SYM-NEXT:     Type: 0x0
----------------
ditto


================
Comment at: llvm/test/CodeGen/PowerPC/aix-tls-le-xcoff-reloc.ll:193
+; SYM-NEXT:     Index: 7
+; SYM-NEXT:     Name: .data
+; SYM-NEXT:     Value (RelocatableAddress): 0x30
----------------
ditto


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149722



More information about the llvm-commits mailing list