From: goar5670 Date: Mon, 6 Mar 2023 22:25:39 +0000 (+0300) Subject: gccrs: typecheck: Add basic typechecking for ReferenceType X-Git-Tag: basepoints/gcc-15~2794 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=5104a38ae8563f011934452dee7f7aa3c89967ea;p=thirdparty%2Fgcc.git gccrs: typecheck: Add basic typechecking for ReferenceType gcc/rust/ChangeLog: * hir/tree/rust-hir-pattern.h (HIR::ReferencePattern): added get_referenced_pattern function. * resolve/rust-ast-resolve-pattern.h (Resolve::PatternDeclaration): add visit function for AST::ReferencePattern * typecheck/rust-hir-type-check-pattern.cc (Resolver::TypeCheckPattern): add visit function for HIR::ReferencePattern gcc/testsuite/ChangeLog: * rust/compile/ref_pattern_fn_param.rs: New test. Signed-off-by: Mahmoud Mohamed --- diff --git a/gcc/rust/hir/tree/rust-hir-pattern.h b/gcc/rust/hir/tree/rust-hir-pattern.h index 5826c8b3ee6f..17921cc0391c 100644 --- a/gcc/rust/hir/tree/rust-hir-pattern.h +++ b/gcc/rust/hir/tree/rust-hir-pattern.h @@ -477,6 +477,12 @@ public: return PatternType::REFERENCE; } + std::unique_ptr &get_referenced_pattern () + { + rust_assert (pattern != nullptr); + return pattern; + } + protected: /* Use covariance to implement clone function as returning this object rather * than base */ diff --git a/gcc/rust/resolve/rust-ast-resolve-pattern.h b/gcc/rust/resolve/rust-ast-resolve-pattern.h index d8de17920053..0811a031386e 100644 --- a/gcc/rust/resolve/rust-ast-resolve-pattern.h +++ b/gcc/rust/resolve/rust-ast-resolve-pattern.h @@ -75,6 +75,11 @@ public: pattern.get_pattern_in_parens ()->accept_vis (*this); } + void visit (AST::ReferencePattern &pattern) override + { + pattern.get_referenced_pattern ()->accept_vis (*this); + } + // cases in a match expression void visit (AST::PathInExpression &pattern) override; diff --git a/gcc/rust/typecheck/rust-hir-type-check-pattern.cc b/gcc/rust/typecheck/rust-hir-type-check-pattern.cc index f5ac7baf51ee..b277a5630fcd 100644 --- a/gcc/rust/typecheck/rust-hir-type-check-pattern.cc +++ b/gcc/rust/typecheck/rust-hir-type-check-pattern.cc @@ -410,8 +410,13 @@ TypeCheckPattern::visit (HIR::QualifiedPathInExpression &pattern) void TypeCheckPattern::visit (HIR::ReferencePattern &pattern) { - rust_sorry_at (pattern.get_locus (), - "type checking qualified path patterns not supported"); + if (parent->get_kind () != TyTy::TypeKind::REF) + rust_error_at (pattern.get_locus (), "expected %s, found reference", + parent->as_string ().c_str ()); + + TyTy::ReferenceType *ref_ty_ty = static_cast (parent); + TypeCheckPattern::Resolve (pattern.get_referenced_pattern ().get (), + ref_ty_ty->get_base ()); } void diff --git a/gcc/testsuite/rust/compile/ref_pattern_fn_param.rs b/gcc/testsuite/rust/compile/ref_pattern_fn_param.rs new file mode 100644 index 000000000000..67ea033a2b23 --- /dev/null +++ b/gcc/testsuite/rust/compile/ref_pattern_fn_param.rs @@ -0,0 +1 @@ +fn f(&b: i32) {} // { dg-error "expected i32, found reference" }