This is an archive of the discontinued LLVM Phabricator instance.

[llvm-nm] Demangle Rust symbols
ClosedPublic

Authored by tmiasko on Oct 16 2021, 5:16 AM.

Details

Summary

Add support for demangling Rust v0 symbols to llvm-nm by reusing
nonMicrosoftDemangle which supports both Itanium and Rust mangling.

Diff Detail

Event Timeline

tmiasko created this revision.Oct 16 2021, 5:16 AM
tmiasko requested review of this revision.Oct 16 2021, 5:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 16 2021, 5:16 AM
dblaikie accepted this revision.Oct 16 2021, 9:22 AM

Looks good

This revision is now accepted and ready to land.Oct 16 2021, 9:22 AM
jhenderson accepted this revision.Oct 18 2021, 12:48 AM

LGTM.

I had some concern about this now meaning some names getting demangled when they shouldn't be, but at the same time, such names would only come from C code, and those names are reserved by the compiler. I guess in such a situation, the user could just not use the --demangle option.

Perhaps worth a release note?

This revision was automatically updated to reflect the committed changes.