[PATCH] D42076: [WebAssembly] Symbol changes #1, LLD

[Nicholas Wilson via Phabricator via llvm-commits]

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>