Skip to content

Disable bounds check for WAMR#22

Open
psilva261 wants to merge 4 commits intomasterfrom
psilva261/wamr-bounds-check
Open

Disable bounds check for WAMR#22
psilva261 wants to merge 4 commits intomasterfrom
psilva261/wamr-bounds-check

Conversation

@psilva261
Copy link
Collaborator

@psilva261 psilva261 commented Feb 12, 2026

  • Export shutdown function as workaround unclean exit issue
  • Reduces no. steps by 35%

Originally --bounds-check=1 was introduced to allow for a clean exit. Since it creates a performance penalty, here is a workaround based on an explicit shutdown call. This seems like a clean option for a quick fix, also since the languages in use do bounds checks anyway.

* Export shutdown function as workaround unclean exit issue
* Reduces no. steps by 35%
@psilva261 psilva261 force-pushed the psilva261/wamr-bounds-check branch from 26968dd to 16ed03e Compare February 12, 2026 14:21
@psilva261 psilva261 marked this pull request as ready for review February 12, 2026 14:21
@github-actions
Copy link

Benchmark Results (825e8f1)

Language native w2c2-O0 w2c2-O3 wamr wasmer wasmtime
stateless 235192341 6025861918 N/A 3526878553 949795341 867158639
reva-client-eth 387697026 4471418839 1375615326 2358970704 N/A 1027868792

@kevaundray
Copy link
Contributor

CC @marcinbugaj

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