Skip to content

Commit

Permalink
Rollup merge of #105162 - compiler-errors:fn-sig-arity, r=cjgillot
Browse files Browse the repository at this point in the history
Properly synthesize `FnSig` value during cycle

Get the arity correct when creating a `FnSig` type during `tcx.fn_sig` cycle recovery

Fixes #105152
  • Loading branch information
matthiaskrgr authored Dec 2, 2022
2 parents de0d18a + 5809a05 commit babdf86
Show file tree
Hide file tree
Showing 3 changed files with 32 additions and 5 deletions.
20 changes: 15 additions & 5 deletions compiler/rustc_middle/src/values.rs
Original file line number Diff line number Diff line change
Expand Up @@ -32,13 +32,23 @@ impl<'tcx> Value<TyCtxt<'tcx>> for ty::SymbolName<'_> {
}

impl<'tcx> Value<TyCtxt<'tcx>> for ty::Binder<'_, ty::FnSig<'_>> {
fn from_cycle_error(tcx: TyCtxt<'tcx>, _: &[QueryInfo]) -> Self {
fn from_cycle_error(tcx: TyCtxt<'tcx>, stack: &[QueryInfo]) -> Self {
let err = tcx.ty_error();
// FIXME(compiler-errors): It would be nice if we could get the
// query key, so we could at least generate a fn signature that
// has the right arity.

let arity = if let Some(frame) = stack.get(0)
&& frame.query.name == "fn_sig"
&& let Some(def_id) = frame.query.def_id
&& let Some(node) = tcx.hir().get_if_local(def_id)
&& let Some(sig) = node.fn_sig()
{
sig.decl.inputs.len() + sig.decl.implicit_self.has_implicit_self() as usize
} else {
tcx.sess.abort_if_errors();
unreachable!()
};

let fn_sig = ty::Binder::dummy(tcx.mk_fn_sig(
[].into_iter(),
std::iter::repeat(err).take(arity),
err,
false,
rustc_hir::Unsafety::Normal,
Expand Down
8 changes: 8 additions & 0 deletions src/test/ui/query-system/fn-sig-cycle-arity.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
trait Dancer {
fn dance(&self) -> _ {
//~^ ERROR the placeholder `_` is not allowed within types on item signatures for return types
self.dance()
}
}

fn main() {}
9 changes: 9 additions & 0 deletions src/test/ui/query-system/fn-sig-cycle-arity.stderr
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
error[E0121]: the placeholder `_` is not allowed within types on item signatures for return types
--> $DIR/fn-sig-cycle-arity.rs:2:24
|
LL | fn dance(&self) -> _ {
| ^ not allowed in type signatures

error: aborting due to previous error

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

0 comments on commit babdf86

Please sign in to comment.