[PATCH] D42117: [WebAssembly] Symbol changes #6: syminfo index, LLVM

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 19 10:56:27 PST 2018


sbc100 added a comment.

In https://reviews.llvm.org/D42117#982103, @ncw wrote:

> Updates:
>
> - Rebased
> - Removed `std::sort()` of the symbol flags, that wasn't actually doing anything useful since obj2yaml reorders them anyway


Hmm.. obj2yaml is sorting stuff ?  I don't think it should since its supposed to mirror the binary directly.


Repository:
  rL LLVM

https://reviews.llvm.org/D42117





More information about the llvm-commits mailing list