Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEAT] Extend the Analyses to other Hard Forks #24

Open
3 tasks
iamrecursion opened this issue May 25, 2023 · 0 comments
Open
3 tasks

[FEAT] Extend the Analyses to other Hard Forks #24

iamrecursion opened this issue May 25, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@iamrecursion
Copy link
Contributor

Description

Currently the storage layout analysis only includes gas costs based on the current hard fork. We want to be able to analyse contracts compiled for other hard forks, and hence should allow specifying differing gas costs based on the fork or chain.

This should be similar to the work for #14.

Spec

  • Add a supportedForks(&self) -> Vec<u8> method to the Opcode trait.
  • Alter the min_gas_cost method to also take a fork_version parameter that lets it return different gas based on the fork it is 'executing' under.
  • Take the fork as a parameter when starting the analysis, and pass it through.
@iamrecursion iamrecursion added the enhancement New feature or request label May 25, 2023
@iamrecursion iamrecursion mentioned this issue May 25, 2023
2 tasks
@iamrecursion iamrecursion changed the title [TASK] Extend the Analyses to other Hard Forks [FEAT] Extend the Analyses to other Hard Forks Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant