[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/tools/llvm-readobj/XCOFFDumper.cpp:463-464
+  Expected<DenseMap<uint64_t, StringRef>> StrTableOrErr = Obj.getStringTable();
+  if (!StrTableOrErr)
+    reportUniqueWarning("unable to get the string table: " +
+                        toString(StrTableOrErr.takeError()));
----------------
You need to add a test case that will exercise this warning. You also need to return after the warning, or you'll attempt to print the table contents when you don't have a valid map!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104613