[PATCH] D44323: [WebAssembly] Remove a second parameter from toString().

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added inline comments.


================
Comment at: wasm/Symbols.cpp:186
     if (Optional<std::string> S = demangleItanium(Sym.getName()))
-      return QuoteDemangled ? ("`" + *S + "'") : *S;
   return Sym.getName();
----------------
I'm ok with this change, but I like this feature, so I would like to bring it back in some form, perhaps for all backends.  Otherwise it makes very long C++ symbols very hard to distinguish from the rest of the error/message.

This is what gnu ld does I believe.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D44323