Skip to content

Commit 6189b26

Browse files
committed
web: propagate arg names from callee JUMPDEST in TraceDrawer
Same fix as the programs-react mockTrace.ts change — the duplicated call stack builder in TraceDrawer.tsx also needs to update argumentNames from the callee entry step, not just argumentPointers.
1 parent 5010254 commit 6189b26

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

packages/web/src/theme/ProgramExample/TraceDrawer.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,8 +140,10 @@ function TraceDrawerContent(): JSX.Element {
140140
if (isDuplicate) {
141141
// Use the callee entry step for resolution —
142142
// argument pointers reference stack slots
143-
// valid at the JUMPDEST, not the JUMP
143+
// valid at the JUMPDEST, not the JUMP.
144+
// Argument names also live on the callee entry.
144145
top.stepIndex = i;
146+
top.argumentNames = info.argumentNames ?? top.argumentNames;
145147
top.argumentPointers = info.argumentPointers;
146148
} else {
147149
frames.push({

0 commit comments

Comments
 (0)