[PATCH] D43675: [WebAssembly] Rename imported/exported memory symbol to __linear_memory

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 2 09:55:07 PST 2018


sbc100 added a comment.

You do make some good points and I agree with most of them.

I would say that it is not just toolchain who will want to do access the wasm memory.   Its basically anyone who wants to write there own JS to load and use wasm module.  If I just want to build my own wasm and then load it myself and call a function I will almost certainly want to read and write data to the wasm's memory.   I'm not saying that is a strong argument for calling it `memory` I'm just saying its not always an internal detail.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D43675





More information about the llvm-commits mailing list