[PATCH] D41038: [WebAssembly] Preserve ordering of global symbols

[Nicholas Wilson via Phabricator via llvm-commits]

ncw accepted this revision.
ncw added a comment.
This revision is now accepted and ready to land.

Looks good. Is the ordering mainly for the stability of the test output, or is it expected to have a tiny improvement to code locality at runtime?

There might be a couple of other places where we could do the same change. For example `calculateImports` has exactly the same iteration pattern, where the imports are ordered according to the first-importing file, rather than ordered by the file that actually uses the import. Harder to see how to fix that one though. I wouldn't have thought it's worth trying.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D41038