Skip to content

propagate opaque types to the typeck root #135445

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
lcnr wants to merge 6 commits into from
Closed

propagate opaque types to the typeck root #135445

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
0848e9e
generalize `ClosureOutlivesSubjectTy`
lcnr Jan 13, 2025
815cf11
propagate opaque types in closures to parent
lcnr Jan 13, 2025
b7f8ceb
HACK: avoid an existing ICE
lcnr Jan 15, 2025
9b3630f
HACK: avoid `give_region_a_name` ICE
lcnr Jan 15, 2025
5cb2d9d
woops
lcnr Jan 15, 2025
417be2c
don't explicitly require `'external: 'body`
lcnr Jan 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
HACK: avoid an existing ICE
  • Loading branch information
@lcnr
lcnr committed Jan 17, 2025
commit b7f8ceb75f4fda67737a9ed720bf34b2513b7516
10 changes: 7 additions & 3 deletions compiler/rustc_middle/src/mir/tcx.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,9 +114,13 @@ impl<'tcx> PlaceTy<'tcx> {
}
let answer = match *elem {
ProjectionElem::Deref => {
let ty = self.ty.builtin_deref(true).unwrap_or_else(|| {
bug!("deref projection of non-dereferenceable ty {:?}", self)
});
let pointee_ty = self.ty;
let ty = pointee_ty
.builtin_deref(true)
.or_else(|| Some(Ty::new_error(tcx, pointee_ty.error_reported().err()?)))
.unwrap_or_else(|| {
bug!("deref projection of non-dereferenceable ty {:?}", self)
});
PlaceTy::from_ty(ty)
}
ProjectionElem::Index(_) | ProjectionElem::ConstantIndex { .. } => {
Expand Down