Initial table.get/set implementation would match and lower combinations
of GEP+load/store to table.get/set instructions. However, this is error
prone due to potential combinations of GEP+load/store we don't implement,
and load/store optimizations. By changing the code to using intrinsics, we
avoid both issues and simplify the code.
New builtins implemented:
- @llvm.wasm.table.get.externref
- @llvm.wasm.table.get.funcref
- @llvm.wasm.table.set.externref
- @llvm.wasm.table.set.funcref
Nit: this line seems to have unwanted whitespace (a bunch of spaces)