search

vyperlang / vyper

Pythonic Smart Contract Language for the EVM
https://vyperlang.org
Other
4.83k stars 789 forks source link

feat[test]: remove memory mocker #4005

Closed charles-cooper closed 3 months ago

charles-cooper commented 3 months ago

it doesn't really find bugs (since the calling convention was redone in 0.3.0) and we have better ways to analyze memory now since the introduction of the venom pipeline.

What I did

How I did it

How to verify it

Commit message

Commit message for the final, squashed PR. (Optional, but reviewers will appreciate it! Please see our commit message style guide for what we would ideally like to see in a commit message.)

Description for the changelog

Cute Animal Picture

![Put a link to a cute animal picture inside the parenthesis-->]()

fubuloubu commented 3 months ago

we have better ways to analyze memory now since the introduction of the venom pipeline

Link?

charles-cooper commented 3 months ago

we have better ways to analyze memory now since the introduction of the venom pipeline

Link?

see for instance https://github.com/vyperlang/vyper/pull/3941, we don't even allocate physical memory locations in the frontend anymore when venom pipeline is enabled. the plan is to do memory allocation as a much later pass in venom after memory analysis has been performed and redundant memory allocations removed.