[PATCH] D43101: [WebAssebmly] Report undefined symbols correctly in objdump

Nicholas Wilson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 9 12:27:18 PST 2018


ncw added a comment.

This looks good, maybe you spotted it could be improved when you saw I'd thoughtlessly copied the current llvm-objdump output for the new data/global symbol split!

There will be conflicts with the symtab work, but not especially hard to resolve. Having said that, I think the conflicts and merge procedure would be rather simpler if the symtab stuff went in first (since it's a bigger thing to rebase and rejig overall).

Edit: d'oh, you merged it literally as I was typing :(


Repository:
  rL LLVM

https://reviews.llvm.org/D43101





More information about the llvm-commits mailing list