[PATCH] D34668: llvm-nm: Add suport for symbol demangling (-C/--demangle)

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

I'm fine with this but I'd appreciate another pair of eyes as it's been a while since I touched the dumpers.



================
Comment at: tools/llvm-nm/llvm-nm.cpp:759
     uint32_t SymFlags;
+    std::string Name = I->Name.str();
+    if (Demangle) {
----------------
Do you need a copy here?


https://reviews.llvm.org/D34668