-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
multiple vmtrace compatibility issues #4071
Comments
there are like 0 docs for all these things so not surprising that things are missing. The missing fields weren't part of the original OE impl. what does |
This issue is stale because it has been open for 14 days with no activity. |
@mattsse Is this resolved via the linked PRs? Going through it seems like it? |
closing this, if there are still issues we track them separately once they get reported |
Describe the bug
these are findings from a brief comparison with erigon. note that erigon's tracing routines are not free from bugs either.
op
is missingidx
is missingmem
values are missingused
should track the remaining gas, currently it's equal tocost
in rethpush
on stack currently uses full words, while in other clients you'd see0xc0ffeebabe
rather than0x000000000000000000000000000000000000000000000000000000c0ffeebabe
code
has an extra zero word at the end for some reasonSteps to reproduce
trace_replayTransaction
with reth and erigon and diff the outputNode logs
No response
Platform(s)
Linux (x86)
What version/commit are you on?
0.1.0-alpha.4
What database version are you on?
No response
If you've built Reth from source, provide the full command you used
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: