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

wasm build error in nightly-2018-05-13 #50731

Closed
CryZe opened this issue May 14, 2018 · 3 comments
Closed

wasm build error in nightly-2018-05-13 #50731

CryZe opened this issue May 14, 2018 · 3 comments
Labels
C-bug Category: This is a bug. I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ O-wasm Target: WASM (WebAssembly), http://webassembly.org/ 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.
Milestone

Comments

@CryZe
Copy link
Contributor

CryZe commented May 14, 2018

Looks like something broke building wasm in this range: ff2ac35...9fae153

This is the error:

thread 'main' panicked at 'discriminant range is 170141183460469231731687303715884105727...-170141183460469231731687303715884105728', librustc/ty/layout.rs:891:17
note: Run with `RUST_BACKTRACE=1` for a backtrace.
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
note: rustc 1.27.0-nightly (9fae15374 2018-05-13) running on x86_64-unknown-linux-gnu
note: compiler flags: -C opt-level=3 -C panic=abort --crate-type lib
note: some of the compiler flags provided by cargo are hidden

https://travis-ci.org/LiveSplit/livesplit-core/jobs/378520318#L551

@CryZe
Copy link
Contributor Author

CryZe commented May 14, 2018

Considering it is caused in layout.rs, it seems to be a bug introduced by #50622 which was beta backported. So if that's it, the bugfix probably needs to be backported too.

@oli-obk
Copy link
Contributor

oli-obk commented May 14, 2018

cc @eddyb

@kennytm kennytm added I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ regression-from-stable-to-nightly Performance or correctness regression from stable to nightly. O-wasm Target: WASM (WebAssembly), http://webassembly.org/ C-bug Category: This is a bug. regression-from-stable-to-beta Performance or correctness regression from stable to beta. and removed regression-from-stable-to-nightly Performance or correctness regression from stable to nightly. labels May 14, 2018
@eddyb
Copy link
Member

eddyb commented May 14, 2018

Minimal testcase:

enum Void {}
fn foo(_: Result<(Void, u32), (Void, String)>) {}
fn main() {
    let _: fn(_) = foo;
}

@pietroalbini pietroalbini added this to the 1.27 milestone May 14, 2018
@pietroalbini pietroalbini added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label May 14, 2018
bors added a commit that referenced this issue May 14, 2018
rustc: don't trip an assertion for enums with present but uninhabited variants.

Fixes #50731.

r? @nikomatsakis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug. I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ O-wasm Target: WASM (WebAssembly), http://webassembly.org/ 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

5 participants