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

Implemented execution/evm.go #58

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from
Draft

Conversation

x-senpai-x
Copy link

CallInner function left to implement I can't figure out how to implement it . Also changed address field of common/types.go

@x-senpai-x
Copy link
Author

@star-gazer111 Although I have written call_inner function but there are several problems:
stateDB which is being used in geth vm package has a newEvm function which uses statedDB however I'll need to implement all the methods in that interface for it to work

@x-senpai-x
Copy link
Author

Also an issue of execution/rpc has not been created till now

@star-gazer111
Copy link
Member

what is the curren status on this @x-senpai-x ?

@x-senpai-x
Copy link
Author

I am working with veer to resolve callInner . I am hoping it will be resolved be tommorow> what is the curren status on this @x-senpai-x ?

@x-senpai-x
Copy link
Author

@star-gazer111 could you please help in callInner function . I am unable to implement this

@gerceboss
Copy link
Contributor

@x-senpai-x ,,state the difficulty you are facing and in implementing what ?

@x-senpai-x
Copy link
Author

@gerceboss in the call inner function when we use vm.NewEvm it takes as arguments BlockContext TxContext StateDB chainConfig and config so the problem is that StateDB here is different from the proofDB that we have implemented in evm.go.
However in rust NewEvm func (of Revm) uses the ProofDB implemented in evm.rs

@x-senpai-x
Copy link
Author

actually what I did was I implemented everything apart from call,callInner and getEnv. But now there is a problem in proofDB that has been created

@gerceboss
Copy link
Contributor

@x-senpai-x Please remove files like .DS_store , go.mod and go.sum and rebase first.

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.

3 participants