Skip to content

Commit

Permalink
Rollup merge of #113651 - lcnr:parent-def-id, r=compiler-errors
Browse files Browse the repository at this point in the history
  • Loading branch information
matthiaskrgr committed Jul 17, 2023
2 parents e31ebae + e449daa commit eca9c01
Show file tree
Hide file tree
Showing 7 changed files with 30 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ use crate::FnCtxt;
use rustc_hir as hir;
use rustc_hir::def::Res;
use rustc_hir::def_id::DefId;
use rustc_infer::traits::ObligationCauseCode;
use rustc_infer::{infer::type_variable::TypeVariableOriginKind, traits::ObligationCauseCode};
use rustc_middle::ty::{self, Ty, TyCtxt, TypeSuperVisitable, TypeVisitable, TypeVisitor};
use rustc_span::{self, symbol::kw, Span};
use rustc_trait_selection::traits;
Expand Down Expand Up @@ -267,8 +267,7 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
type BreakTy = ty::GenericArg<'tcx>;
fn visit_ty(&mut self, ty: Ty<'tcx>) -> std::ops::ControlFlow<Self::BreakTy> {
if let Some(origin) = self.0.type_var_origin(ty)
&& let rustc_infer::infer::type_variable::TypeVariableOriginKind::TypeParameterDefinition(_, def_id) =
origin.kind
&& let TypeVariableOriginKind::TypeParameterDefinition(_, def_id) = origin.kind
&& let generics = self.0.tcx.generics_of(self.1)
&& let Some(index) = generics.param_def_id_to_index(self.0.tcx, def_id)
&& let Some(subst) = ty::GenericArgs::identity_for_item(self.0.tcx, self.1)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ fn fmt_printer<'a, 'tcx>(infcx: &'a InferCtxt<'tcx>, ns: Namespace) -> FmtPrinte
let ty_vars = infcx_inner.type_variables();
let var_origin = ty_vars.var_origin(ty_vid);
if let TypeVariableOriginKind::TypeParameterDefinition(name, def_id) = var_origin.kind
&& !var_origin.span.from_expansion()
&& name != kw::SelfUpper && !var_origin.span.from_expansion()
{
let generics = infcx.tcx.generics_of(infcx.tcx.parent(def_id));
let idx = generics.param_def_id_to_index(infcx.tcx, def_id).unwrap();
Expand Down
11 changes: 4 additions & 7 deletions compiler/rustc_trait_selection/src/traits/error_reporting/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2388,14 +2388,11 @@ impl<'tcx> InferCtxtPrivExt<'tcx> for TypeErrCtxt<'_, 'tcx> {
// If there is only one implementation of the trait, suggest using it.
// Otherwise, use a placeholder comment for the implementation.
let (message, impl_suggestion) = if non_blanket_impl_count == 1 {(
"use the fully-qualified path to the only available implementation".to_string(),
"use the fully-qualified path to the only available implementation",
format!("<{} as ", self.tcx.type_of(impl_def_id).instantiate_identity())
)} else {(
format!(
"use a fully-qualified path to a specific available implementation ({} found)",
non_blanket_impl_count
),
"</* self type */ as ".to_string()
)} else {
("use a fully-qualified path to a specific available implementation",
"</* self type */ as ".to_string()
)};
let mut suggestions = vec![(
path.span.shrink_to_lo(),
Expand Down
2 changes: 1 addition & 1 deletion tests/ui/error-codes/E0283.stderr
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ LL | fn create() -> u32;
LL | let cont: u32 = Generator::create();
| ^^^^^^^^^^^^^^^^^ cannot call associated function of trait
|
help: use a fully-qualified path to a specific available implementation (2 found)
help: use a fully-qualified path to a specific available implementation
|
LL | let cont: u32 = </* self type */ as Generator>::create();
| +++++++++++++++++++ +
Expand Down
2 changes: 1 addition & 1 deletion tests/ui/error-codes/E0790.stderr
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ LL | fn my_fn();
LL | MyTrait2::my_fn();
| ^^^^^^^^^^^^^^^ cannot call associated function of trait
|
help: use a fully-qualified path to a specific available implementation (2 found)
help: use a fully-qualified path to a specific available implementation
|
LL | </* self type */ as MyTrait2>::my_fn();
| +++++++++++++++++++ +
Expand Down
7 changes: 7 additions & 0 deletions tests/ui/inference/need_type_info/infer-var-for-self-param.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
// Regression test for #113610 where we ICEd when trying to print
// inference variables created by instantiating the self type parameter.

fn main() {
let _ = (Default::default(),);
//~^ ERROR cannot call associated function on trait
}
14 changes: 14 additions & 0 deletions tests/ui/inference/need_type_info/infer-var-for-self-param.stderr
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
error[E0790]: cannot call associated function on trait without specifying the corresponding `impl` type
--> $DIR/infer-var-for-self-param.rs:5:14
|
LL | let _ = (Default::default(),);
| ^^^^^^^^^^^^^^^^ cannot call associated function of trait
|
help: use a fully-qualified path to a specific available implementation
|
LL | let _ = (</* self type */ as Default>::default(),);
| +++++++++++++++++++ +

error: aborting due to previous error

For more information about this error, try `rustc --explain E0790`.

0 comments on commit eca9c01

Please sign in to comment.