[PATCH] D42282: [WebAssembly] Update to match llvm change to SYM_INFO section

Nicholas Wilson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 19 01:50:43 PST 2018


ncw added a comment.

I suppose this //could// be merged, since it doesn't make things worse. LLD `--relocatable` output is already broken for aliases, but it doesn't feel great to be introducing another place where it comes unstuck.

At the very least, it needs a comment added to the `getOutputIndex()` call saying `// TODO fixme once we have symbol indexes`.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D42282





More information about the llvm-commits mailing list