wasmtime_benchmark fix #278
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
TRSWNCA
reviewed
Sep 9, 2025
Collaborator
There was a problem hiding this comment.
- 直接写需要安装的内容和设置的环境变量
- build.sh通过参数的形式简化操作，不需要用户去修改build.sh
- 能否把相同语言的build.sh统一了
- 写到justfile里，不要用原始手动的形式去让用户测试
TRSWNCA
reviewed
Sep 9, 2025
scripts/build_big_data_wasm.sh
Outdated
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
修改了wasmtime_wordcount 和 wasmtime_parallel_sort 工作流测试C代码，修复了部分bug，修改了wasm编译脚本，支持用户自定义给宏传参。修改数据生成脚本gen_data.py，通过固定种子，保证相同大小的数据集生成的数据一致。目前wasmtime_wordcount_c1、c3、c5可自定义参数并顺利运行， wasmtime_parallel_sort_c1可顺利运行并时间上有优化，c3、c5并行运行测试未成功，有待修改。