This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Reorder exports to match symbol table order
ClosedPublic

Authored by ncw on Feb 28 2018, 9:50 AM.

Event Timeline

ncw created this revision.Feb 28 2018, 9:50 AM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 1 2018, 1:40 AM
This revision was automatically updated to reflect the committed changes.
ncw added a comment.Mar 1 2018, 1:43 AM

Merged this one as well since it's pretty routine. I hope that's OK procedurally, for PRs with few/minimal code changes!

sbc100 added a comment.Mar 1 2018, 8:11 AM

LGTM.

I think I'd prefer all but the most trivial changes to go through code review if thats OK.

ncw added a comment.Mar 1 2018, 8:21 AM

OK, I'll wait for review for non-trivial code changes then. Thanks!