[PATCH] D89049: [AIX][XCOFF] print out the traceback info

Digger via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 22 09:19:29 PST 2020


DiggerLin marked 4 inline comments as done.
DiggerLin added inline comments.


================
Comment at: llvm/tools/llvm-objdump/XCOFFDump.cpp:114
+
+std::string AddOffsetToValueInString(const std::string &StrMsg,
+                                     unsigned Offset) {
----------------
jhenderson wrote:
> Can this function be static too?
I still use the const std::string &StrMsg as parameter here, for   int Num = std::stoi(NumString, 0, 16) and Res += std::string(StrMsg, PrePos, Pos - PrePos);  we still need to convert the StringRef to std:string.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89049



More information about the llvm-commits mailing list