[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