Skip to content

Commit

Permalink
fix: Wrong BoundVar index when lowering impl trait parameter of paren…
Browse files Browse the repository at this point in the history
…t generics
  • Loading branch information
ShoyuVanilla committed Aug 17, 2024
1 parent 0765978 commit c1618ad
Show file tree
Hide file tree
Showing 2 changed files with 50 additions and 18 deletions.
35 changes: 17 additions & 18 deletions crates/hir-ty/src/lower.rs
Original file line number Diff line number Diff line change
Expand Up @@ -377,26 +377,25 @@ impl<'a> TyLoweringContext<'a> {
// Count the number of `impl Trait` things that appear within our bounds.
// Since t hose have been emitted as implicit type args already.
counter.set(idx + count_impl_traits(type_ref) as u16);
let (
_parent_params,
self_param,
type_params,
const_params,
_impl_trait_params,
lifetime_params,
) = self
let kind = self
.generics()
.expect("variable impl trait lowering must be in a generic def")
.provenance_split();
TyKind::BoundVar(BoundVar::new(
self.in_binders,
idx as usize
+ self_param as usize
+ type_params
+ const_params
+ lifetime_params,
))
.intern(Interner)
.iter()
.enumerate()
.filter_map(|(i, (id, data))| match (id, data) {
(
GenericParamId::TypeParamId(_),
GenericParamDataRef::TypeParamData(data),
) if data.provenance == TypeParamProvenance::ArgumentImplTrait => {
Some(i)
}
_ => None,
})
.nth(idx as usize)
.map_or(TyKind::Error, |id| {
TyKind::BoundVar(BoundVar { debruijn: self.in_binders, index: id })
});
kind.intern(Interner)
}
ImplTraitLoweringState::Disallowed => {
// FIXME: report error
Expand Down
33 changes: 33 additions & 0 deletions crates/hir-ty/src/tests/regression.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2162,3 +2162,36 @@ fn main() {
"#]],
);
}

#[test]
fn issue_17711() {
check_infer(
r#"
//- minicore: deref
use core::ops::Deref;
struct Struct<'a, T>(&'a T)
trait Trait {}
impl<'a, T: Deref<Target = impl Trait>> Struct<'a, T> {
fn foo(&self) -> &Self { self }
fn bar(&self) {
let _ = self.foo();
}
}
"#,
expect![[r#"
136..140 'self': &'? Struct<'a, T>
151..159 '{ self }': &'? Struct<'a, T>
153..157 'self': &'? Struct<'a, T>
173..177 'self': &'? Struct<'a, T>
179..214 '{ ... }': ()
193..194 '_': &'? Struct<'?, T>
197..201 'self': &'? Struct<'a, T>
197..207 'self.foo()': &'? Struct<'?, T>
"#]],
);
}

0 comments on commit c1618ad

Please sign in to comment.