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

Normalize types before using them in debuginfo. #33099

Merged
merged 1 commit into from
Apr 26, 2016

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Apr 19, 2016

Small oversight, fixes #33096 - odd thing is that the old code doesn't look like it should've ever worked, although it wasn't using all of the type parameters, so maybe that's what changed.

@rust-highfive
Copy link
Collaborator

r? @nrc

(rust_highfive has picked a reviewer for you, use r? to override)

@nrc
Copy link
Member

nrc commented Apr 19, 2016

r? @michaelwoerister

@michaelwoerister
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Apr 20, 2016

📌 Commit 4e1cf9a has been approved by michaelwoerister

@bors
Copy link
Contributor

bors commented Apr 22, 2016

⌛ Testing commit 4e1cf9a with merge 71b1cb6...

@bors
Copy link
Contributor

bors commented Apr 22, 2016

💔 Test failed - auto-win-msvc-64-opt-mir

@alexcrichton
Copy link
Member

@bors: retry

On Fri, Apr 22, 2016 at 8:31 AM, bors [email protected] wrote:

[image: 💔] Test failed - auto-win-msvc-64-opt-mir
http://buildbot.rust-lang.org/builders/auto-win-msvc-64-opt-mir/builds/406


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#33099 (comment)

@bors
Copy link
Contributor

bors commented Apr 22, 2016

⌛ Testing commit 4e1cf9a with merge e983071...

bors added a commit that referenced this pull request Apr 22, 2016
Normalize types before using them in debuginfo.

Small oversight, fixes #33096 - odd thing is that the old code doesn't look like it should've ever worked, although it wasn't using all of the type parameters, so maybe that's what changed.
@bors
Copy link
Contributor

bors commented Apr 22, 2016

💔 Test failed - auto-win-msvc-32-opt

@alexcrichton
Copy link
Member

@bors: retry

On Fri, Apr 22, 2016 at 11:38 AM, bors [email protected] wrote:

[image: 💔] Test failed - auto-win-msvc-32-opt
http://buildbot.rust-lang.org/builders/auto-win-msvc-32-opt/builds/3118


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#33099 (comment)

@bors
Copy link
Contributor

bors commented Apr 23, 2016

⌛ Testing commit 4e1cf9a with merge dfa4c85...

@bors
Copy link
Contributor

bors commented Apr 23, 2016

💔 Test failed - auto-linux-64-opt-rustbuild

@eddyb
Copy link
Member Author

eddyb commented Apr 25, 2016

@bors retry

@bors
Copy link
Contributor

bors commented Apr 25, 2016

⌛ Testing commit 4e1cf9a with merge d52627a...

@bors
Copy link
Contributor

bors commented Apr 25, 2016

⛄ The build was interrupted to prioritize another pull request.

@bors
Copy link
Contributor

bors commented Apr 26, 2016

⌛ Testing commit 4e1cf9a with merge 82b7e4d...

@bors
Copy link
Contributor

bors commented Apr 26, 2016

⛄ The build was interrupted to prioritize another pull request.

@bors
Copy link
Contributor

bors commented Apr 26, 2016

⌛ Testing commit 4e1cf9a with merge 442038a...

@bors
Copy link
Contributor

bors commented Apr 26, 2016

⛄ The build was interrupted to prioritize another pull request.

@bors
Copy link
Contributor

bors commented Apr 26, 2016

⌛ Testing commit 4e1cf9a with merge 769ac42...

bors added a commit that referenced this pull request Apr 26, 2016
Normalize types before using them in debuginfo.

Small oversight, fixes #33096 - odd thing is that the old code doesn't look like it should've ever worked, although it wasn't using all of the type parameters, so maybe that's what changed.
@bors bors merged commit 4e1cf9a into rust-lang:master Apr 26, 2016
@eddyb eddyb deleted the issue-33096 branch April 26, 2016 10:37
@michaelwoerister
Copy link
Member

🎉

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.

Compilation ICE on nphysics crate
6 participants