Skip to content

S-expression parsing in C API and binaryen.js#1050

Merged
kripken merged 8 commits intomasterfrom
c-s-parse
Jun 14, 2017
Merged

S-expression parsing in C API and binaryen.js#1050
kripken merged 8 commits intomasterfrom
c-s-parse

Conversation

@kripken
Copy link
Copy Markdown
Member

@kripken kripken commented Jun 10, 2017

BinaryenModuleParse and Binaryen.parseText respectively.

@kripken kripken mentioned this pull request Jun 10, 2017
@dcodeIO
Copy link
Copy Markdown
Contributor

dcodeIO commented Jun 10, 2017

Is there a chance to support this on function / source code level, too? For example, I have a couple of builtins that I am currently declaring programmatically, but it would be super convenient to be able to use some sort of inline-assembly instead. More a nice-to-have, not super important, of course.

Could be that it is already possible to emulate this somehow with this PR when using temporary modules. Iirc, there is already some debugging utility that does the inverse, i.e. converting a subset of statements or expressions to s-expr syntax. Not sure if that's exposed already, though.

@kripken
Copy link
Copy Markdown
Member Author

kripken commented Jun 10, 2017

Yeah, this PR gets us pretty close to that. I think we'd just need to add module merging to get what you want. We already have wasm-merge as a tool, it could be refactored into a C API call.

@kripken kripken merged commit 8aa91ec into master Jun 14, 2017
@kripken kripken deleted the c-s-parse branch June 14, 2017 04:04
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.

2 participants