Add ICE regression test with unboxed closures #17093

Merged
merged 1 commit into from Sep 9, 2014

Projects

None yet

4 participants

@nathantypanski
Contributor

This code used to produce the following ICE:

error: internal compiler error: get_unique_type_id_of_type() -
unexpected type: closure,
ty_unboxed_closure(syntax::ast::DefId{krate: 0u32, node: 66u32},
ReScope(63u32))

This is a regression test for issue #17021.

@nathantypanski nathantypanski Add ICE regression test with unboxed closures
This code used to produce the following ICE:

   error: internal compiler error: get_unique_type_id_of_type() -
   unexpected type: closure,
   ty_unboxed_closure(syntax::ast::DefId{krate: 0u32, node: 66u32},
   ReScope(63u32))

This is a regression test for issue #17021.
a1d9010
@nrc
nrc commented on a1d9010 Sep 8, 2014

r+

@brson
Contributor
brson commented Sep 8, 2014

<3 tests

@alexcrichton alexcrichton added a commit to alexcrichton/rust that referenced this pull request Sep 9, 2014
@alexcrichton alexcrichton rollup merge of #17093 : nathantypanski/17021 3039d0d
@alexcrichton alexcrichton added a commit to alexcrichton/rust that referenced this pull request Sep 9, 2014
@alexcrichton alexcrichton rollup merge of #17093 : nathantypanski/17021 e561a28
@bors bors merged commit a1d9010 into rust-lang:master Sep 9, 2014

1 check failed

continuous-integration/travis-ci The Travis CI build failed
Details
@nathantypanski nathantypanski added a commit to nathantypanski/rust that referenced this pull request Sep 12, 2014
@nathantypanski nathantypanski add debuginfo for unboxed closures
This catches two unhandled patterns in middle/trans/debuginfo.rs where a
ty::ty_unboxed_closure could show up, preventing various potential ICEs
when #[feature(unboxed_closures)] is used with `-g`.

It also updates the test case from #17093 in response to new information
from #17021 showing that the test still caused an ICE when compiling
with debuginfo.

Closes #17021.
8861e1d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment