[PATCH] D42076: [WebAssembly] Symbol changes #1, LLD
Nicholas Wilson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 15 09:07:48 PST 2018
ncw created this revision.
ncw added a reviewer: sbc100.
Herald added subscribers: llvm-commits, sunfish, aheejin, jgravelle-google, dschuff, jfb.
First chunk split out of https://reviews.llvm.org/D41955.
Must be committed simultaneously with LLD change, https://reviews.llvm.org/D42075.
Changes:
- Update for DEBUG_FUNCTION_NAME disappearing
- Simplify generation of "names" section by simply iterating over the DefinedFunctions array.
- Apparently, this even fixes some bugs, judging by the test changes required - some tests are asserting that functions are named multiple times, other tests are asserting that the "names" section contains the function's alias rather than its original name...
- Rolled in some tiny formatting changes...
Repository:
rLLD LLVM Linker
https://reviews.llvm.org/D42076
Files:
test/wasm/alias.ll
test/wasm/archive.ll
test/wasm/relocatable.ll
test/wasm/weak-symbols.ll
wasm/InputChunks.h
wasm/InputFiles.cpp
wasm/Symbols.h
wasm/Writer.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42076.129874.patch
Type: text/x-patch
Size: 11403 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180115/11d7ba4a/attachment.bin>
More information about the llvm-commits
mailing list