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

Pattern referring to nonexistent variant causes ICE on beta. #34929

Closed
eddyb opened this issue Jul 20, 2016 · 3 comments
Closed

Pattern referring to nonexistent variant causes ICE on beta. #34929

eddyb opened this issue Jul 20, 2016 · 3 comments
Labels
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.

Comments

@eddyb
Copy link
Member

eddyb commented Jul 20, 2016

enum E { V { x: u8 } }

fn main() {
    match (E::V { x: 0 }) {
        E::V_Typoed { .. } => {}
    }
}

Produces, on beta:

error: internal compiler error: ../src/librustc/hir/def.rs:77: path not fully resolved:
PathResolution { base_def: Enum(DefId { krate: 0, node: DefIndex(4) => E }), depth: 1 }

On stable and nightly it triggers an appropriate resolution failure error. cc @petrochenkov

@eddyb eddyb 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 Jul 20, 2016
@Aatch
Copy link
Contributor

Aatch commented Jul 20, 2016

Be nice to know what change fixed it again so we can backport it into beta.

@petrochenkov
Copy link
Contributor

Be nice to know what change fixed it again so we can backport it into beta.

It's a397b60, I'll extract the minimal fix for beta.

@alexcrichton
Copy link
Member

Fixed in #34964

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) ❄️ regression-from-stable-to-beta Performance or correctness regression from stable to beta.
Projects
None yet
Development

No branches or pull requests

4 participants