diff options
| author | Mazdak Farrokhzad <twingoow@gmail.com> | 2019-07-04 01:38:44 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-07-04 01:38:44 +0200 |
| commit | 6cfd474e3337934da9606844e6cf571f96d9652b (patch) | |
| tree | 50f291bba07aadff790bbf6d17ce8010cabc101f | |
| parent | 740d5bd1575b52d501d3431a82bb326469d93b00 (diff) | |
| parent | a02d4364e47724ee342c2fdac09a38877c20a984 (diff) | |
| download | rust-6cfd474e3337934da9606844e6cf571f96d9652b.tar.gz rust-6cfd474e3337934da9606844e6cf571f96d9652b.zip | |
Rollup merge of #62240 - arielb1:resolve-wf-fields, r=pnkfelix
wfcheck: resolve the type-vars in `AdtField` types Normalization can leave some type-vars unresolved in its return type. Make sure to resolve them so we have an infcx-independent type that can be used with `needs_drop`. Fixes #61402. Closes #62212 - this PR fixes the root cause.
| -rw-r--r-- | src/librustc_typeck/check/wfcheck.rs | 2 | ||||
| -rw-r--r-- | src/test/run-pass/packed/packed-with-inference-vars-issue-61402.rs | 21 |
2 files changed, 23 insertions, 0 deletions
diff --git a/src/librustc_typeck/check/wfcheck.rs b/src/librustc_typeck/check/wfcheck.rs index 2c48bd8b4f0..3aa144ca352 100644 --- a/src/librustc_typeck/check/wfcheck.rs +++ b/src/librustc_typeck/check/wfcheck.rs @@ -1104,6 +1104,8 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> { let field_ty = self.tcx.type_of(self.tcx.hir().local_def_id_from_hir_id(field.hir_id)); let field_ty = self.normalize_associated_types_in(field.span, &field_ty); + let field_ty = self.resolve_vars_if_possible(&field_ty); + debug!("non_enum_variant: type of field {:?} is {:?}", field, field_ty); AdtField { ty: field_ty, span: field.span } }) .collect(); diff --git a/src/test/run-pass/packed/packed-with-inference-vars-issue-61402.rs b/src/test/run-pass/packed/packed-with-inference-vars-issue-61402.rs new file mode 100644 index 00000000000..6028b8f5ab4 --- /dev/null +++ b/src/test/run-pass/packed/packed-with-inference-vars-issue-61402.rs @@ -0,0 +1,21 @@ +// If a struct is packed and its last field has drop glue, then that +// field needs to be Sized (to allow it to be destroyed out-of-place). +// +// This is checked by the compiler during wfcheck. That check used +// to have problems with associated types in the last field - test +// that this doesn't ICE. + +#![allow(unused_imports, dead_code)] + +pub struct S; + +pub trait Trait<R> { type Assoc; } + +impl<X> Trait<X> for S { type Assoc = X; } + +#[repr(C, packed)] +struct PackedAssocSized { + pos: Box<<S as Trait<usize>>::Assoc>, +} + +fn main() { println!("Hello, world!"); } |
