[PATCH] D45021: [WebAssembly] Distinguish debug/symbol names in the Wasm structs. NFC

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added a comment.

One comment, otherwise lgtm



================
Comment at: include/llvm/BinaryFormat/Wasm.h:104
+  StringRef SymbolName; // from the "linking" section
+  StringRef DebugName; // from the "name" section
   uint32_t Comdat; // from the "comdat info" section
----------------
What do you think about dropping DebugName completely here?  Since the name section is not really used in the object format right now this would make sense to me.  If you really want to get to the debug names we have a method on the object to get them right?


Repository:
  rL LLVM

https://reviews.llvm.org/D45021