[PATCH] D101140: [WebAssembly][CodeGen] IR support for WebAssembly local variables

Thomas Lively via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 31 21:59:46 PDT 2021


tlively accepted this revision.
tlively added a comment.

In D101140#2789199 <https://reviews.llvm.org/D101140#2789199>, @wingo wrote:

> Quick r? to @tlively -- OK with having the new util function in WebAssemblyFrameLowering?  It didn't work to have it in the utils library because the function needs the vtable of WebAssemblyFunctionInfo.

Yes, this sounds reasonable to me.

And regarding the target-specific additions to target-independent code, this addition also seems similar to how we have target-specific MVTs in the target-independent MVT list. Being able to add names (but usually not logic) for target-specific concepts to target-independent namespaces makes it much simpler to handle target-independent and target-specific things in a uniform way in the backends.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101140



More information about the cfe-commits mailing list