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

Nicholas Wilson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 1 09:28:05 PST 2018


ncw added a comment.

I guess the multi-step process would be:

1. Add commandline option allowing LLD's name for the memory import to be changed
2. Emscripten passes on the option to request the new symbol name, and fixes whatever needs to be fixed
3. Then we change the default and remove the commandline option

Seems a bit laborious, but if it's necessary then it's necessary. I don't really have a say on the process... for merging it, whatever the Emscripten devs want really.

Basically I'm not in a position to merge this LLD change; someone from Emscripten would need to nice and coordinate :) I'm just trying to make it easy-ish for someone with Emscripten commit access to have a go at making the change.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D43675





More information about the llvm-commits mailing list