Skip to content
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

debuginfo: Work around crash-bug in MSDIA library #42803

Merged
merged 1 commit into from Jun 22, 2017

Conversation

michaelwoerister
Copy link
Member

Fixes #40477 (which also contains a description of the issue being fixed).

r? @vadimcn

@vadimcn
Copy link
Contributor

vadimcn commented Jun 21, 2017

Can you please add some examples of problematic signatures and what they get transformed to in the comment? Is it something like fn(a:[(); 10]) -> fn(a: *const ())?

Otherwise, LGTM.
r=me

@michaelwoerister
Copy link
Member Author

Can you please add some examples of problematic signatures and what they get transformed to in the comment?

Excellent idea, will do.

@michaelwoerister
Copy link
Member Author

@bors r=vadimcn

Added examples to the comment.

@bors
Copy link
Contributor

bors commented Jun 22, 2017

📌 Commit 8261413 has been approved by vadimcn

@bors
Copy link
Contributor

bors commented Jun 22, 2017

⌛ Testing commit 8261413 with merge 03c8b92...

bors added a commit that referenced this pull request Jun 22, 2017
debuginfo: Work around crash-bug in MSDIA library

Fixes #40477 (which also contains a description of the issue being fixed).

r? @vadimcn
@bors
Copy link
Contributor

bors commented Jun 22, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: vadimcn
Pushing 03c8b92 to master...

@bors bors merged commit 8261413 into rust-lang:master Jun 22, 2017
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.

For function signatures with arrays, the generated PDB file crashes MSVC's msdia DLL
3 participants