pmatos added inline comments.
================ Comment at: llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp:1138 + + Ops[0] = TableSet; // The new chain is the TableSet itself + } ---------------- sbc100 wrote: > I was expecting to see a TABLE_SET back to a null after the call here to > avoid the GC root being leaked in table slot 0 (as above). I must be > missing something? @sbc100, that's what's happening in line 583. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104797/new/ https://reviews.llvm.org/D104797 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
