[PATCH] D104613: [llvm-readobj][XCOFF] Add support for printing the String Table.
Xing GUO via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 21 19:33:40 PDT 2021
Higuoxing added inline comments.
================
Comment at: llvm/lib/Object/XCOFFObjectFile.cpp:190
+Expected<DenseMap<uint64_t, StringRef>>
+XCOFFObjectFile::getStringTable() const {
----------------
Since we are modifying `lib/Object/XCOFFObjectFile.cpp` and this function may return an error, I think we need to write unittest for this function. What do you think, @jhenderson ?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104613/new/
https://reviews.llvm.org/D104613
More information about the llvm-commits
mailing list