Skip to content

Add --stack-maps to wasmtime objdump#10439

Merged
alexcrichton merged 2 commits intobytecodealliance:mainfrom
alexcrichton:objdump-stack-maps
Mar 20, 2025
Merged

Add --stack-maps to wasmtime objdump#10439
alexcrichton merged 2 commits intobytecodealliance:mainfrom
alexcrichton:objdump-stack-maps

Conversation

@alexcrichton
Copy link
Member

Follow-up to #10404 and #10405

@alexcrichton alexcrichton requested a review from a team as a code owner March 20, 2025 20:36
@alexcrichton alexcrichton requested review from fitzgen and removed request for a team March 20, 2025 20:36
Comment on lines +66 to +68
/// Whether or not to show information about stack maps.
#[arg(long)]
stack_maps: bool,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one also seems like it should probably be on by default and won't be too noisy or anything like that.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok I've done some rejiggering of the types and such for clap and I think I got something working, namely --traps turns things on, --traps=false turns things off, and it's on-by-default.

@alexcrichton alexcrichton enabled auto-merge March 20, 2025 21:09
@alexcrichton alexcrichton added this pull request to the merge queue Mar 20, 2025
Merged via the queue into bytecodealliance:main with commit 5b07689 Mar 20, 2025
40 checks passed
@alexcrichton alexcrichton deleted the objdump-stack-maps branch March 20, 2025 21:45
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.

2 participants