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

Rust 1.18 ICE regression: assertion failed: !unsize (nightly regression #41604) #41944

Closed
KamilaBorowska opened this issue May 12, 2017 · 8 comments
Labels
I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ P-high High priority regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@KamilaBorowska
Copy link
Contributor

KamilaBorowska commented May 12, 2017

When compiling image with beta, the following error happens.

error: internal compiler error: unexpected panic
note: the compiler unexpectedly panicked. this is a bug.
note: we would appreciate a bug report: https://github.com/rust-lang/rust/blob/master/CONTRIBUTING.md#bug-reports
thread 'rustc' panicked at 'assertion failed: !unsize', /checkout/src/librustc_typeck/check/mod.rs:2185
note: Run with `RUST_BACKTRACE=1` for a backtrace.
error: Could not compile `image`.

This was reported in nightly as #41604 and fixed in nightly. Reported by https://users.rust-lang.org/t/how-to-report-problems-with-rust-beta/10759?u=xfix.

@nagisa nagisa added I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ regression-from-stable-to-beta Performance or correctness regression from stable to beta. labels May 12, 2017
@johnthagen
Copy link
Contributor

@Mark-Simulacrum
Copy link
Member

I'm confused. As far as I can tell, this is a known problem -- #41604 -- and is awaiting a backport to beta, which would then fix it there. Shouldn't this be closed?

@johnthagen
Copy link
Contributor

@Mark-Simulacrum I'm totally fine with closing if it's a known, solved issue. I just hadn't seen that fix after looking around and wanting to err on the side of reporting.

@Mark-Simulacrum
Copy link
Member

Oh, yeah, please do report, it's better to have the report than not have it and miss a regression. However, since I know a lot of people hit this (both on nightly and beta) I'll keep this open to allow them to land here. We can close it in the potential future beta backport PR.

wahn added a commit to wahn/rs_pbrt that referenced this issue May 15, 2017
@nagisa nagisa added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. P-high High priority labels May 18, 2017
@michaelwoerister
Copy link
Member

Once #42053 lands, this should also be fixed on beta.

@brson
Copy link
Contributor

brson commented May 22, 2017

Also affects life-0.1.2 cc @lj-ditrapani

@nikomatsakis
Copy link
Contributor

I think I have verified that life-0.1.2 is fixed by #42053. Though I wonder if I've messed up my testing setup.

@brson
Copy link
Contributor

brson commented May 25, 2017

Thanks @nikomatsakis. I've also confirmed that life is fixed on beta.

@brson brson closed this as completed May 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ P-high High priority regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

7 participants