wasm-instrument

pezkuwichain/wasm-instrument

Fork 0

mirror of https://github.com/pezkuwichain/wasm-instrument.git synced 2026-06-13 19:51:08 +00:00

Commit Graph

Author	SHA1	Message	Date
StackOverflowExcept1on	fdce5c64f1	chore: modernize WASM tests (#79 ) I tried locally merge `fix-tests` and `fix-clippy` branches and CI should work after my PRs	2023-11-07 09:14:09 +01:00
Sergey Shulepov	19760b5835	Fix counting of the local variables The code assumed that the number of `Local` and number of locals is the same thing. In reality though it is not. `Local` actually represents a group of locals with the same type. The group can declare more than one variable and the number of them is returned by `Local::count`. In this PR we acknowledge this fact. Along the way we add a checked arithmetic for locals_count and max_stack_height summation.	2021-06-09 19:22:40 +02:00

Author

SHA1

Message

Date

StackOverflowExcept1on

fdce5c64f1

chore: modernize WASM tests (#79 )

I tried locally merge `fix-tests` and `fix-clippy` branches and CI
should work after my PRs

2023-11-07 09:14:09 +01:00

Sergey Shulepov

19760b5835

Fix counting of the local variables

The code assumed that the number of `Local` and number of locals is the
same thing. In reality though it is not. `Local` actually represents a
group of locals with the same type. The group can declare more than one
variable and the number of them is returned by `Local::count`.

In this PR we acknowledge this fact. Along the way we add a checked
arithmetic for locals_count and max_stack_height summation.

2021-06-09 19:22:40 +02:00

2 Commits