Skip to content

wasm2js2 import fixes#2031

Merged
kripken merged 2 commits intomasterfrom
wasm2js2
Apr 19, 2019
Merged

wasm2js2 import fixes#2031
kripken merged 2 commits intomasterfrom
wasm2js2

Conversation

@kripken
Copy link
Copy Markdown
Member

@kripken kripken commented Apr 19, 2019

  • Don't assume function types exist in legalize-js-interface.
  • Properly handle (ignore) imports in RemoveNonJSOps - do not try to recurse into them.
  • Run legalize-js-interface and remove-unused-module-elements in wasm2js, the first is necessary, the last is nice to have.

@kripken kripken merged commit db14d04 into master Apr 19, 2019
@kripken kripken deleted the wasm2js2 branch April 19, 2019 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant