[lld] [llvm] [WebAssembly] Use 64-bit table when targeting wasm64 (PR #92042)

Sam Clegg via llvm-commits llvm-commits at lists.llvm.org
Mon May 13 16:44:31 PDT 2024


sbc100 wrote:

> > It still only takes a `table32_op` operand.. Am I missing something?
> > https://github.com/llvm/llvm-project/blob/23f8fac745bdde70ed4f9c585d19c4913734f1b8/llvm/lib/Target/WebAssembly/WebAssemblyInstrCall.td#L61-L67
> 
> I guess that code might need updating too.

Actually isn't `table32_op` here referring to the table immediate?  i.e. the index of the table itself, not the index into the table which is taken as an argument?   That fact that its called `$table` (and that the tests pass) makes me think this.

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


More information about the llvm-commits mailing list