[PATCH] D90608: [WebAssembly] Implement ref.null

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added a comment.

lgtm.   In a project that primarily deals with C/C++ code I find "heap time" a little confusing because it makes me think of the malloc heap used by C/C++.   Why not "managed" or something like that to avoid ambiguity?   I guess that is being discussed in the spec and we should probably just follow their lead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90608