Skip to content

Fix symbol demangling on Windows. #19819

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Dec 18, 2014
Merged

Fix symbol demangling on Windows. #19819

merged 2 commits into from
Dec 18, 2014

Conversation

vadimcn
Copy link
Contributor

@vadimcn vadimcn commented Dec 13, 2014

Windows dbghelp strips leading underscores from symbols, and I could not find a way to turn this off. So let's accept "ZN...E" form too.
Also, print PC displacement from symbols. This is helpful in gauging whether the PC was indeed within the function displayed in the backtrace, or whether it just happened to be the closest public symbol in the module.

…ept "ZN...E" form too.

Also, print PC displacement from symbols.
bors added a commit that referenced this pull request Dec 14, 2014
Fix symbol demangling on Windows.

Reviewed-by: alexcrichton
@brson
Copy link
Contributor

brson commented Dec 16, 2014

@vadimcn
Copy link
Contributor Author

vadimcn commented Dec 17, 2014

re-r?

bors added a commit that referenced this pull request Dec 18, 2014
Windows dbghelp strips leading underscores from symbols, and I could not find a way to turn this off.  So let's accept  "ZN...E" form too.
Also, print PC displacement from symbols.  This is helpful in gauging whether the PC was indeed within the function displayed in the backtrace, or whether it just happened to be the closest public symbol in the module.
@bors bors merged commit b3b7185 into rust-lang:master Dec 18, 2014
@vadimcn vadimcn deleted the fix-demangle branch March 1, 2015 07:59
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.

4 participants