[PATCH] D90948: [WebAssembly] call_indirect issues table number relocs

Fri Feb 19 05:50:45 PST 2021

wingo marked an inline comment as done.
wingo added inline comments.

================
Comment at: llvm/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp:611
+              WasmSym->setType(SymbolType.getValue());
+            }
+          }
----------------
wingo wrote:
> tlively wrote:
> > wingo wrote:
> > > sbc100 wrote:
> > > > I guess it makes sense to piggy back on the existing symbol handling at the end of the instruction...  but would having the symbol immediate come before the signature make more sense for this instruction?
> > > I wasn't sure, tbh, and in the end went with the order in which the fields appear in the binary: https://webassembly.github.io/reference-types/core/binary/instructions.html#control-instructions.  If you think I should change it, happy to do so.
> > In the text format the table index appears first. I think it would be good to be consistent with that rather than the binary format.
> Sure.
Just to confirm, we want the text format to have a different operand order than the CALL_INDIRECT instruction from WebAssemblyCall.td -- can do.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90948