Skip to content

fix: demangle symbols with nm#61

Merged
mkroening merged 2 commits intomainfrom
nm-demangle-instructions
Feb 10, 2026
Merged

fix: demangle symbols with nm#61
mkroening merged 2 commits intomainfrom
nm-demangle-instructions

Conversation

@jounathaen
Copy link
Copy Markdown
Member

@jounathaen jounathaen commented Feb 6, 2026

No description provided.

@mkroening mkroening self-assigned this Feb 10, 2026
@mkroening mkroening force-pushed the nm-demangle-instructions branch from d65779d to eaa4bf1 Compare February 10, 2026 14:23
jounathaen and others added 2 commits February 10, 2026 15:31
Co-authored-by: Martin Kröning <martin.kroening@eonerc.rwth-aachen.de>
@mkroening mkroening force-pushed the nm-demangle-instructions branch from 22245b8 to 2213574 Compare February 10, 2026 14:32
Copy link
Copy Markdown
Member

@mkroening mkroening left a comment

Choose a reason for hiding this comment

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

Thanks! I also applied this to CI and more places across the codebase.

This could potentially lead to errors when the binary is mostly C++ code, but this is rftrace, not cppfrace, right 😉

Just --demangle instead of --demangle=rust works for me too and should automatically detect the mangling format.

This is also more flexible. As of rust-lang/rust@683dd08, the Rust compiler defaults to the v0 Symbol Format for nightly toolchains only, though.

I also applied this to kernel CI: hermit-os/kernel#2243

Diff

@mkroening mkroening added this pull request to the merge queue Feb 10, 2026
@mkroening mkroening removed this pull request from the merge queue due to a manual request Feb 10, 2026
@mkroening mkroening changed the title added demangle flag to symbol generation message fix: demangle symbols with nm Feb 10, 2026
@mkroening mkroening added this pull request to the merge queue Feb 10, 2026
Merged via the queue into main with commit 86a9dc7 Feb 10, 2026
4 checks passed
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