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

Fri Feb 19 06:38:12 PST 2021

wingo added inline comments.

================
Comment at: llvm/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp:611
+              WasmSym->setType(SymbolType.getValue());
+            }
+          }
----------------
wingo wrote:
> 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.
Hummm, I will have to write a dedicated printer in WebAssemblyInstrPrint.cpp for `{RET_,}CALL_INDIRECT_S`, to invert the operand order.  Really sorry to bother but @tlively can you confirm this is the right thing?  Thanks :)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90948