[PATCH] D58132: lld/elf: When demangling symbols, surround demangled symbol with quotes and include mangled symbol as well

Wed Feb 13 11:50:28 PST 2019

thakis added a comment.

In D58132#1396724 <https://reviews.llvm.org/D58132#1396724>, @pcc wrote:

> My personal opinion: 99% of the time I reckon users will want to see the demangled name (it's only us toolchain folks who might more often prefer the mangled names), and there's already a flag `--no-demangle` which can be used to suppress demangling if necessary. So I think I'd prefer making the COFF linker more like the ELF linker (i.e. don't show mangled names without a flag).

Makes sense.

I kind of like the `"demangled" (mangled)` output, but I suppose I can just make that a feature in demumble.

Does lld-link have a --no-demangle flag already? If not, any opinions on how it should be spelled?

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

https://reviews.llvm.org/D58132