-
Notifications
You must be signed in to change notification settings - Fork 12.6k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Rollup merge of #102016 - lcnr:given-OutlivesEnvironment, r=jackh726
implied_bounds: deal with inference vars fixes #101951 while computing implied bounds for `<<T as ConstructionFirm>::Builder as BuilderFn<'_>>::Output` normalization replaces a projection with an inference var (adding a `Projection` obligation). Until we prove that obligation, this inference var remains unknown, which caused us to miss an implied bound necessary to prove that the unnormalized projection from the trait method signature is wf. r? types
- Loading branch information
Showing
5 changed files
with
91 additions
and
34 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,50 @@ | ||
// Taken directly from that issue. | ||
// | ||
// This test detected that we didn't correctly resolve | ||
// inference variables when computing implied bounds. | ||
// | ||
// check-pass | ||
pub trait BuilderFn<'a> { | ||
type Output; | ||
} | ||
|
||
impl<'a, F, Out> BuilderFn<'a> for F | ||
where | ||
F: FnOnce(&'a mut ()) -> Out, | ||
{ | ||
type Output = Out; | ||
} | ||
|
||
pub trait ConstructionFirm { | ||
type Builder: for<'a> BuilderFn<'a>; | ||
} | ||
|
||
pub trait Campus<T> | ||
where | ||
T: ConstructionFirm, | ||
{ | ||
fn add_building( | ||
&mut self, | ||
building: &mut <<T as ConstructionFirm>::Builder as BuilderFn<'_>>::Output, | ||
); | ||
} | ||
|
||
struct ArchitectsInc {} | ||
|
||
impl ConstructionFirm for ArchitectsInc { | ||
type Builder = fn(&mut ()) -> PrettyCondo<'_>; | ||
} | ||
|
||
struct PrettyCondo<'a> { | ||
_marker: &'a mut (), | ||
} | ||
|
||
struct CondoEstate {} | ||
|
||
impl Campus<ArchitectsInc> for CondoEstate { | ||
fn add_building(&mut self, _building: &mut PrettyCondo<'_>) { | ||
todo!() | ||
} | ||
} | ||
|
||
fn main() {} |