[PATCH] D109595: [WebAssembly] Convert to new "dylink.0" section format

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added inline comments.


================
Comment at: llvm/tools/obj2yaml/wasm2yaml.cpp:54
   std::unique_ptr<WasmYAML::CustomSection> CustomSec;
-  if (WasmSec.Name == "dylink") {
+  if (WasmSec.Name == "dylink" || WasmSec.Name == "dylink.0") {
     std::unique_ptr<WasmYAML::DylinkSection> DylinkSec =
----------------
dschuff wrote:
> So wasm2yaml will accept either name. Does work because the parsing goes through lib/Object, which knows how to handle the legacy section format?
Yes version are supported for now.  And yes `lib/Object` hides the differences and exposes the same struct whichever version is used.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109595