Skip to content

Require _Z* when doing C++ demangling - #14599

Merged
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:demangle-less
Oct 7, 2026
Merged

alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:demangle-less

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

Don't demangle function names like i as int.

Closes #4162

Don't demangle function names like `i` as `int`.

Closes bytecodealliance#4162
@alexcrichton
alexcrichton requested a review from a team as a code owner October 6, 2026 23:29
@alexcrichton
alexcrichton requested review from pchickey and removed request for a team October 6, 2026 23:29

@fitzgen fitzgen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we want something similar for Rust symbols?

@alexcrichton

Copy link
Copy Markdown
Member Author

My understanding (which could very well be wrong) is that rustc_demangle basically already does this and it's basically C++-quirks reasons why cpp_demangle doesn't.

@alexcrichton
alexcrichton enabled auto-merge October 6, 2026 23:36
@alexcrichton
alexcrichton added this pull request to the merge queue Oct 6, 2026
Merged via the queue into bytecodealliance:main with commit 87bb9aa Oct 7, 2026
53 checks passed
@alexcrichton
alexcrichton deleted the demangle-less branch October 7, 2026 00:02
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.

Wasmtime is too eager to use C++ name demangling

2 participants