[PATCH] D37886: [WebAssembly] Add support for naming wasm data segments

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 15 16:05:10 PDT 2017


sbc100 added a comment.

Yes, there is the potential that we might need more metadata down the road (type, flags, etc) but right now I think name is enough for to get init_array and fini_array working.

WRT to merging I plan to follow the same rules as the ELF version of lld.  I'm hoping that name will be enough for me to do this, but I guess I might need type/flags down the road.   However this change should be enough to get init_array / fini_array working which is my immediate goal.


https://reviews.llvm.org/D37886





More information about the llvm-commits mailing list