Skip to content

core,eth: call frame tracing (#23087)#30

Merged
uprendis merged 1 commit intoFantom-foundation:develop-1.10.8from
uprendis:develop-1.10.8-txtracing
Mar 4, 2022
Merged

core,eth: call frame tracing (#23087)#30
uprendis merged 1 commit intoFantom-foundation:develop-1.10.8from
uprendis:develop-1.10.8-txtracing

Conversation

@quan8
Copy link
Copy Markdown

@quan8 quan8 commented Mar 2, 2022

This change introduces 2 new optional methods; enter() and exit() for js tracers, and makes step() optiona. The two new methods are invoked when entering and exiting a call frame (but not invoked for the outermost scope, which has it's own methods). Currently these are the data fields passed to each of them:

enter: type (opcode), from, to, input, gas, value
exit: output, gasUsed, error

The PR also comes with a re-write of the callTracer. As a backup we keep the previous tracing script under the name callTracerLegacy. Behaviour of both tracers are equivalent for the most part, although there are some small differences (improvements), where the new tracer is more correct / has more information.

This change introduces 2 new optional methods; `enter()` and `exit()` for js tracers, and makes `step()` optiona. The two new methods are invoked when entering and exiting a call frame (but not invoked for the outermost scope, which has it's own methods). Currently these are the data fields passed to each of them:

    enter: type (opcode), from, to, input, gas, value
    exit: output, gasUsed, error

The PR also comes with a re-write of the callTracer. As a backup we keep the previous tracing script under the name `callTracerLegacy`. Behaviour of both tracers are equivalent for the most part, although there are some small differences (improvements), where the new tracer is more correct / has more information.
@quan8 quan8 changed the base branch from v1.10.8-ftm-rc4 to develop-1.10.8 March 3, 2022 17:26
@uprendis uprendis merged commit 778b950 into Fantom-foundation:develop-1.10.8 Mar 4, 2022
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