---

yaml
---
r: 164615
b: refs/heads/auto
c: b3dcb85
h: refs/heads/master
i:
  164613: 5a2ee10
  164611: 1050bbe
  164607: 224fe03
v: v3

Author: nikomatsakis

[refs]

@@ -10,7 +10,7 @@ refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503
 refs/tags/release-0.3.1: 495bae036dfe5ec6ceafd3312b4dca48741e845b
 refs/tags/release-0.4: e828ea2080499553b97dfe33b3f4d472b4562ad7
 refs/tags/release-0.5: 7e3bcfbf21278251ee936ad53e92e9b719702d73
-refs/heads/auto: 1b24602ca083cc20169190556a51066af0438049
+refs/heads/auto: b3dcb85404aabd66a5765bb99aa46e24acede4dc
 refs/heads/servo: af82457af293e2a842ba6b7759b70288da276167
 refs/tags/release-0.6: b4ebcfa1812664df5e142f0134a5faea3918544c
 refs/tags/0.1: b19db808c2793fe2976759b85a355c3ad8c8b336

branches/auto/src/librustc_typeck/coherence/mod.rs

@@ -18,6 +18,7 @@
 
 use metadata::csearch::{each_impl, get_impl_trait};
 use metadata::csearch;
+use middle::region;
 use middle::subst::{mod, Subst};
 use middle::ty::{ImplContainer, ImplOrTraitItemId, MethodTraitItemId};
 use middle::ty::{ParameterEnvironment, TypeTraitItemId, lookup_item_type};
@@ -26,8 +27,6 @@ use middle::ty::{ty_param, Polytype, ty_ptr};
 use middle::ty::{ty_rptr, ty_struct, ty_trait, ty_tup};
 use middle::ty::{ty_str, ty_vec, ty_float, ty_infer, ty_int, ty_open};
 use middle::ty::{ty_uint, ty_unboxed_closure, ty_uniq, ty_bare_fn};
-use middle::ty::{ty_closure};
-use middle::subst::Subst;
 use middle::ty;
 use CrateCtxt;
 use middle::infer::combine::Combine;
@@ -472,6 +471,17 @@ impl<'a, 'tcx> CoherenceChecker<'a, 'tcx> {
                                                            impl_did.node);
             let self_type = self_type.ty.subst(tcx, &param_env.free_substs);
 
+            // the self-type may have late-bound regions bound in the
+            // impl; liberate them.
+            let item_scope = region::CodeExtent::from_node_id(impl_did.node);
+            let self_type =
+                ty::liberate_late_bound_regions(tcx,
+                                                item_scope,
+                                                &ty::bind(self_type)).value;
+
+            debug!("can_type_implement_copy(self_type={})",
+                   self_type.repr(tcx));
+
             match ty::can_type_implement_copy(tcx, self_type, &param_env) {
                 Ok(()) => {}
                 Err(ty::FieldDoesNotImplementCopy(name)) => {

branches/auto/src/test/run-pass/hrtb-opt-in-copy.rs

@@ -0,0 +1,38 @@
+// Copyright 2014 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 <LICENSE-APACHE or
+// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
+// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
+// option. This file may not be copied, modified, or distributed
+// except according to those terms.
+
+// Test that we handle binder levels correctly when checking whether a
+// type can implement `Copy`. In particular, we had a bug where we failed to
+// liberate the late-bound regions from the impl, and thus wound up
+// searching for an impl of `for<'tcx> Foo<&'tcx T>`. The impl that
+// exists however is `impl<T> Copy for Foo<T>` and the current rules
+// did not consider that a match (something I would like to revise in
+// a later PR).
+
+#![allow(dead_code)]
+
+use std::kinds::marker;
+
+#[deriving(Copy)]
+struct Foo<T> { x: T }
+
+type Ty<'tcx> = &'tcx TyS<'tcx>;
+
+enum TyS<'tcx> {
+    Boop(marker::InvariantLifetime<'tcx>)
+}
+
+enum Bar<'tcx> {
+    Baz(Foo<Ty<'tcx>>)
+}
+
+impl<'tcx> Copy for Bar<'tcx> { }
+
+fn main() { }