[lld] [llvm] [WebAssembly] Use ValType instead of integer types to model wasm tables (PR #78012)

Sam Clegg via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 14 17:31:52 PST 2024


================
@@ -258,7 +258,7 @@ static wasm::WasmLimits readLimits(WasmObjectFile::ReadContext &Ctx) {
 
 static wasm::WasmTableType readTableType(WasmObjectFile::ReadContext &Ctx) {
   wasm::WasmTableType TableType;
-  TableType.ElemType = readUint8(Ctx);
+  TableType.ElemType = wasm::ValType(readVaruint32(Ctx));
----------------
sbc100 wrote:

this changes from uint8 to uint32.  Does that match the spec?

https://github.com/llvm/llvm-project/pull/78012


More information about the llvm-commits mailing list