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

VM/EVM: More Code-Separation for Verkle Logic #3576

Closed
holgerd77 opened this issue Aug 9, 2024 · 1 comment
Closed

VM/EVM: More Code-Separation for Verkle Logic #3576

holgerd77 opened this issue Aug 9, 2024 · 1 comment

Comments

@holgerd77
Copy link
Member

Verkle logic is now very broadly integrated into VM/EVM with a lot of in-code 6800-switches and then doing Verkle logic, in EVM in particular all the accessWitness tracking.

Do not have the full picture yet and also not yet a working/practical structural approach, but it would be nice if we would better separate here, for some general better code structuring (this is getting some spaghetti-code tendencies) as well as (optimally) for tree shaking, both within the EVM (so that the EVM(VM) verkle code just is not getting in for non-Verkle usages.

But also - likely bigger - also for the state manager scope with the VerkleStateManager and AccessWitness now always getting in also for all non-Verkle usages.

Open for some rounds of discussion and brainstorming. This also likely need deep looks into the code and some solid thinking about structures.

@holgerd77
Copy link
Member Author

Will close here, no follow-up respectively additional ideas represented, also still not fully sure if practical. Would nevertheless be pretty valuable if someone side-stumble upon ideas on this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant