[PATCH] D41955: [WebAssembly] Symbol changes #5: add "symbol index" concept (LLD)

[Nicholas Wilson via Phabricator via llvm-commits]

ncw updated this revision to Diff 129980.
ncw retitled this revision from "[WebAssembly] Refactor symbol index handling (LLD)" to "[WebAssembly] Symbol changes #5: add "symbol index" concept (LLD)".
ncw edited the summary of this revision.
ncw added a comment.

Updated: split up the big review into six smaller ones.

The two big "problems" described above are now fixed. Problem #1 (stack pointer incompatible with `--emit-relocs` is fixed by using `--relocatable` for that test instead. Problem #2 (taking the address of a local function) is/was fixed in https://reviews.llvm.org/D42105.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D41955

Files:
  test/wasm/archive.ll
  test/wasm/data-layout.ll
  test/wasm/stack-pointer.ll
  test/wasm/weak-alias.ll
  wasm/InputFiles.cpp
  wasm/InputFiles.h
  wasm/Symbols.cpp
  wasm/Symbols.h
  wasm/Writer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41955.129980.patch
Type: text/x-patch
Size: 21171 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180116/d2bede2a/attachment.bin>