0xMiden / 0xMiden/miden-debug

Attribute inlined ops to original procedures in `OpHistogramProc`

Open
#113 0 comments 0 reactions 1 assignee Claimed by @mooori View on GitHub
profiling
Dominant language
Rust
Stars
1
Forks
8
Avg merge
2d 13h
Merged PRs (30d)
16

Description

`OpHistogramProc` currently groups operations using `DebugExecutor::current_proc`, which does not recognize inlining context. For example, when `f1` is inlined into `f2`, operations originating from `f1` are attributed to `f2` in the per-procedure histogram.

@djolertrk is working on preserving inline context in the debug information. When available, use that information in `OpHistogramProc` to attribute inlined operations to their original procedure.

Contributor guide

Open the contributing guide

Research direction

Look at the `OpHistogramProc` implementation to see how it currently uses `DebugExecutor::current_proc`. Examine the debug information work by @djolertrk to understand the inlining context data. Update the histogram logic to attribute operations to the original procedure when inline context is available. Test the changes to ensure operations are correctly attributed after inlining.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
devtools
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.