pharo-vm icon indicating copy to clipboard operation
pharo-vm copied to clipboard

VM Debugger improvement with IR

Open QDucasse opened this issue 3 years ago • 3 comments

Improvement of the VM machine code debugger with the following changes:

  • Automated refresh of the columns (no need to click to refresh)
  • Added abstract instructions in a column on the left
  • Clicking on an abstract instruction shows the corresponding machine code instruction(s)
  • Clicking on a machine code instruction shows the corresponding affected register(s)

Note that the mc <-> registers highlights have been tried with ARMv8 and RISC-V but could be applied to any other ISA as long as the register aliases are correctly defined.

QDucasse avatar Oct 13 '21 11:10 QDucasse

Cool, this is so nice

tesonep avatar Oct 14 '21 09:10 tesonep

I've changed the base branch to Pharo X (and we should work in there in the future). I'll check the conflict.

guillep avatar Oct 14 '21 11:10 guillep

I have the feeling that we will need to manually test after the merge :/. I've made some non-obvious merges.

guillep avatar Oct 14 '21 11:10 guillep

I confirm this is working on the latest version.

imagen

I will test with a more complex method IR and push it if ok!

guillep avatar May 12 '23 07:05 guillep

It seems that

  • we have the stack and and registers panes duplicated
  • the IR <-> machine code mapping is not working in the UI

guillep avatar May 12 '23 07:05 guillep

I've made a pass, this is how it looks now:

imagen

guillep avatar May 12 '23 07:05 guillep

I'll merge it, thanks @QDucasse !!!!

guillep avatar May 12 '23 07:05 guillep