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

Shoaib Meenai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 13 11:49:59 PST 2019


smeenai 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).


I'd be happy with this (just demangled names being the default, as long as there's some way to either switch to or display mangled names in addition).


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

https://reviews.llvm.org/D58132





More information about the llvm-commits mailing list