[PATCH] D115327: [WebAssembly] Fix reftype load/store match with idx from call

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 8 06:50:06 PST 2021


sbc100 added inline comments.


================
Comment at: llvm/test/CodeGen/WebAssembly/externref-tableset.ll:82
 
+declare i32 @get_table_id() local_unnamed_addr
+
----------------
Is this returning the ID of a table, or the ID of slot within a table?  I seem like its the latter, in which case maybe its name is a little confusing?  How about calling this `get_table_slot` ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115327



More information about the llvm-commits mailing list