[PATCH] D103486: [lld][WebAssemlby] Fix for string merging of -dwarf-5 sections

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added inline comments.


================
Comment at: lld/wasm/InputFiles.cpp:375
+  return sec.Name == ".debug_str" || sec.Name == ".debug_str.dwo" ||
+         sec.Name == ".debug_line_str";
 }
----------------
dschuff wrote:
> I guess there is no .debug_line_str.dwo because the `.debug_line` section goes into the main executable, instead of a dwo file?
Indeed.  At least there is no such section marked in `initWasmMCObjectFileInfo` which is the canonical list of string merge-able sections (which is why I mention it here).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103486