[PATCH] D88815: Added .tabletype to asm and multiple table support in obj files

[Sam Clegg via Phabricator via llvm-commits]

sbc100 accepted this revision.
sbc100 added inline comments.


================
Comment at: llvm/include/llvm/BinaryFormat/Wasm.h:73
   WasmLimits Limits;
+  uint32_t Index;
 };
----------------
For other module elements that include `Index` it looks like it is the first elem in the struct (see `Event` and `Function` below)


================
Comment at: llvm/lib/ObjectYAML/WasmYAML.cpp:305
   IO.mapRequired("Limits", Table.TableLimits);
+  IO.mapRequired("Index", Table.Index);
 }
----------------
Move this so its first?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88815