Skip to content

Conversation

@ahuoguo
Copy link
Contributor

@ahuoguo ahuoguo commented Oct 31, 2024

It seems like the .bin.wast generated by wasm -d script.wast -o script.bin.wast removes custom section by default

For wasmfx-tools to remove custom section, you need to do wasmfx-tools strip -a

@ahuoguo ahuoguo changed the base branch from zdh/try-test-spec to main October 31, 2024 02:53
@ahuoguo ahuoguo marked this pull request as ready for review October 31, 2024 02:54
@ahuoguo ahuoguo changed the title basic .bin.wast working (note I changed 0x1 to 1) Basic (module bin ...) support Oct 31, 2024
@ahuoguo ahuoguo changed the base branch from main to zdh/try-test-spec October 31, 2024 02:55
@ahuoguo ahuoguo changed the title Basic (module bin ...) support [WIP] Basic (module bin ...) support Oct 31, 2024
Base automatically changed from zdh/try-test-spec to main November 4, 2024 09:14
@ahuoguo ahuoguo changed the title [WIP] Basic (module bin ...) support Basic (module bin ...) support Nov 4, 2024
@ahuoguo ahuoguo merged commit 2eb36f0 into main Nov 4, 2024
1 check passed
@Kraks Kraks deleted the try-test-spec-bin branch November 4, 2024 23:56
Kraks added a commit that referenced this pull request Nov 18, 2024
* check in MiniWasmFX

* Basic `(module bin ...)` support (#62)


* basic .bin.wast working

---------

Co-authored-by: butterunderflow <azhong.934@gmail.com>
Co-authored-by: Guannan Wei <wei220@purdue.edu>

* refactor value hierarchy

* remove duplication

* prelim impl of RefFunc and ContNew

* callref

* refactor to 2cps style

* add handler

* resumable try-catch

* fix for deep handler

* mechanize test

* add test case

* missing file

* rename vars in resume, added some comments

* comment tests for now

---------

Co-authored-by: ahuoguo <52595524+ahuoguo@users.noreply.github.com>
Co-authored-by: butterunderflow <azhong.934@gmail.com>
Co-authored-by: ahuoguo <ahuoguo@gmail.com>
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.

3 participants