Add MVP of virtual machine tracer - #4931
Conversation
80c0e21 to
2778f0a
Compare
2778f0a to
01e41cd
Compare
Test262 conformance changes
Tested main commit: |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4931 +/- ##
===========================================
+ Coverage 47.24% 62.81% +15.56%
===========================================
Files 476 537 +61
Lines 46892 60433 +13541
===========================================
+ Hits 22154 37958 +15804
+ Misses 24738 22475 -2263 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
|
||
| /// A message that emits instruction execution details about a call frame | ||
| #[derive(Debug, Clone)] | ||
| pub struct OpcodeExecutionMessage { |
There was a problem hiding this comment.
It might be better to granularize this, because some tracers would need to have the individual operands and stack values instead of a big string; I'm thinking of like a web-based tracer that displays the stack in a more dynamic way, for example.
There was a problem hiding this comment.
Ah, sorry just saw this!
Thats a good idea. I'll make the update ASAP
|
Any updates on this? |
|
Still planning to eventually take another pass at this when I have a bit of time. |
Test262 conformance changes
Tested main commit: |
This pull request implements a new
VirtualMachineTracertrait that emitsVirtualMachineEventsrather than immediately printing the values to stdout.This should allow better customization of event handling.