From 5218c2d5efbb2907b279e627782e96896308d106 Mon Sep 17 00:00:00 2001 From: Michael Woerister Date: Wed, 7 Mar 2018 15:24:50 +0100 Subject: [PATCH] Properly handle collecting default impls of methods with lifetime parameters. --- src/librustc_mir/monomorphize/collector.rs | 8 ++++-- src/test/compile-fail/issue-47309.rs | 31 ++++++++++++++++++++++ 2 files changed, 37 insertions(+), 2 deletions(-) create mode 100644 src/test/compile-fail/issue-47309.rs diff --git a/src/librustc_mir/monomorphize/collector.rs b/src/librustc_mir/monomorphize/collector.rs index d43da5392ab8..446ef6bd3287 100644 --- a/src/librustc_mir/monomorphize/collector.rs +++ b/src/librustc_mir/monomorphize/collector.rs @@ -1068,7 +1068,6 @@ fn create_mono_items_for_default_impls<'a, 'tcx>(tcx: TyCtxt<'a, 'tcx, 'tcx>, def_id_to_string(tcx, impl_def_id)); if let Some(trait_ref) = tcx.impl_trait_ref(impl_def_id) { - let callee_substs = tcx.erase_regions(&trait_ref.substs); let overridden_methods: FxHashSet<_> = impl_item_refs.iter() .map(|iiref| iiref.name) @@ -1082,10 +1081,15 @@ fn create_mono_items_for_default_impls<'a, 'tcx>(tcx: TyCtxt<'a, 'tcx, 'tcx>, continue; } + let substs = Substs::for_item(tcx, + method.def_id, + |_, _| tcx.types.re_erased, + |def, _| trait_ref.substs.type_for_def(def)); + let instance = ty::Instance::resolve(tcx, ty::ParamEnv::reveal_all(), method.def_id, - callee_substs).unwrap(); + substs).unwrap(); let mono_item = create_fn_mono_item(instance); if mono_item.is_instantiable(tcx) diff --git a/src/test/compile-fail/issue-47309.rs b/src/test/compile-fail/issue-47309.rs new file mode 100644 index 000000000000..7141bd62dc0d --- /dev/null +++ b/src/test/compile-fail/issue-47309.rs @@ -0,0 +1,31 @@ +// Copyright 2018 The Rust Project Developers. See the COPYRIGHT +// file at the top-level directory of this distribution and at +// http://rust-lang.org/COPYRIGHT. +// +// Licensed under the Apache License, Version 2.0 or the MIT license +// , at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +// Make sure that the mono-item collector does not crash when trying to +// instantiate a default impl of a method with lifetime parameters. +// See https://github.com/rust-lang/rust/issues/47309 + +// compile-flags:-Clink-dead-code +// must-compile-successfully + +#![crate_type="rlib"] + +pub trait EnvFuture { + type Item; + + fn boxed_result<'a>(self) where Self: Sized, Self::Item: 'a, { + } +} + +struct Foo; + +impl<'a> EnvFuture for &'a Foo { + type Item = (); +}