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

Nicholas Wilson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 23 07:14:06 PST 2018


ncw created this revision.
ncw added reviewers: sbc100, sunfish.
Herald added subscribers: llvm-commits, aheejin, jgravelle-google, dschuff, jfb.

It should use a reserved name, rather than one which can collide with a user-defined symbol.

@sunfish, is there anything that would need to change on the Emscripten side to support this?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D43675

Files:
  test/wasm/alias.ll
  test/wasm/call-indirect.ll
  test/wasm/comdats.ll
  test/wasm/entry.ll
  test/wasm/export.ll
  test/wasm/import-memory.test
  test/wasm/load-undefined.test
  test/wasm/local-symbols.ll
  test/wasm/locals-duplicate.test
  test/wasm/visibility-hidden.ll
  test/wasm/weak-alias-overide.ll
  test/wasm/weak-alias.ll
  test/wasm/weak-symbols.ll
  test/wasm/weak-undefined.ll
  wasm/Writer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43675.135635.patch
Type: text/x-patch
Size: 8584 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180223/cfd0471d/attachment.bin>


More information about the llvm-commits mailing list