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

Mon Nov 2 12:52:26 PST 2020

tlively added a comment.

In D90608#2369305 <https://reviews.llvm.org/D90608#2369305>, @sbc100 wrote:

> 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.

I do think that matching the upstream spec "heap type" terminology is important, even if it's a little confusing in a C++ context.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90608