[PATCH] D104613: [llvm-readobj][XCOFF] Add support for printing the String Table.

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: llvm/test/tools/yaml2obj/XCOFF/basic-doc64.yaml:137
+# CHECK64-NEXT: StringTable {
+# CHECK64-NEXT:   [     3]   ..text
+# CHECK64-NEXT:   [     a]   .abs
----------------
Does this test pass? Why is there a second "." in this name, if so?

What's at bytes 0-2 in the string table?


================
Comment at: llvm/test/tools/yaml2obj/XCOFF/long-symbol-name.yaml:27
+# CHECK-NEXT: StringTable {
+# CHECK-NEXT:   [     3]   ..longname
+# CHECK-NEXT: }
----------------
Ditto.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104613