-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Compilation ICE on nphysics crate #33096
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
Comments
cc @eddyb |
You sure it's that innocent commit and not, you know, my changes to debuginfo, for MIR? |
@trolleyman I can't reproduce on master (just rebased one of my branches containing unrelated changes, so I had a compiler to check with). EDIT: Still no reproduction, even after |
Ok so I was misinterpreting which crate was being compiled. nphysics compiles fine, but my crate, which uses nphysics, doesn't. I'm trying to narrow down the code now. |
Here's some of the rust log that I got when compiling on debug builds:
|
@trolleyman That's not really helpful, sadly, nightly doesn't use |
@trolleyman This bug is not in beta, correct? |
@eddyb I'm afraid that I can't test that as I'm using |
@trolleyman Can you check the test from #33099 instead? Particularly interested about the nightlies you had issues with. |
@eddyb Sure, unfortunately that test case works fine for both pre and post- |
@trolleyman Ah, beats me then. |
@eddyb I'll try and find which part of my code hits the error. |
@eddyb So I did some digging and I was wrong again, it turns out it is the nphysics crate. This is a minimal test case:
|
@trolleyman It depends on the |
@eddyb True, sorry but I am not knowledgeable at all in rustc or nphysics, so it's the best I can do :/ |
triage: P-high |
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.
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.
Still need a good regression test. |
triage: P-low |
I'm going to go ahead and close this. It's somewhat unlikely we'll hit this again (at least, we haven't obviously done so in the past 3ish years), and coming up with a good test case when the current case is all of nphysics is likely to be quite hard. |
When trying to compile nphysics on a nightly created on or after
2016-04-14
, the compiler errors. Before this date it is fine.Example code:
Presumably was this commit that broke things, as it's a commit on that day that touches trans.
On Windows 64 bit GNU this is the output:
The text was updated successfully, but these errors were encountered: