]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
gccrs: Support fully qualified calls to trait items from CallExpr's
authorPhilip Herron <herron.philip@googlemail.com>
Tue, 4 Apr 2023 14:40:26 +0000 (15:40 +0100)
committerArthur Cohen <arthur.cohen@embecosm.com>
Tue, 16 Jan 2024 17:28:46 +0000 (18:28 +0100)
Fixes #2070

gcc/rust/ChangeLog:

* backend/rust-compile-resolve-path.cc (HIRCompileBase::query_compile):
reuse resolve_method_address to monomorphize the call if required
* typecheck/rust-hir-type-check-path.cc (TypeCheckExpr::resolve_segments):
allow matching against unimplemented trait item bounds

gcc/testsuite/ChangeLog:

* rust/compile/issue-2070.rs: New test.

Signed-off-by: Philip Herron <herron.philip@googlemail.com>
gcc/rust/backend/rust-compile-resolve-path.cc
gcc/rust/typecheck/rust-hir-type-check-path.cc
gcc/testsuite/rust/compile/issue-2070.rs [new file with mode: 0644]

index 6c9f80edec305caff097ade24d897cee8646bf7d..cbe2f67303aa149aadcf465ec5b88be197c64e34 100644 (file)
@@ -186,6 +186,7 @@ HIRCompileBase::query_compile (HirId ref, TyTy::BaseType *lookup,
     = ctx->get_mappings ()->lookup_hir_extern_item (ref, &parent_block);
   bool is_hir_item = resolved_item != nullptr;
   bool is_hir_extern_item = resolved_extern_item != nullptr;
+  bool is_fn = lookup->get_kind () == TyTy::TypeKind::FNDEF;
   if (is_hir_item)
     {
       if (!lookup->has_subsititions_defined ())
@@ -206,6 +207,21 @@ HIRCompileBase::query_compile (HirId ref, TyTy::BaseType *lookup,
     }
   else
     {
+      if (is_fn)
+       {
+         TyTy::FnType *fn = static_cast<TyTy::FnType *> (lookup);
+         TyTy::BaseType *receiver = nullptr;
+
+         if (fn->is_method ())
+           {
+             receiver = fn->get_self_type ();
+             receiver = receiver->destructure ();
+
+             return resolve_method_address (fn, ref, receiver, final_segment,
+                                            mappings, expr_locus);
+           }
+       }
+
       HirId parent_impl_id = UNKNOWN_HIRID;
       HIR::ImplItem *resolved_item
        = ctx->get_mappings ()->lookup_hir_implitem (ref, &parent_impl_id);
index 322d72a883bda47293e1148ee27e228ec231acd5..c24a1bd37cdd4a66a4f362b7fe7385ddc98636a9 100644 (file)
@@ -312,21 +312,19 @@ TypeCheckExpr::resolve_segments (NodeId root_resolved_node_id,
   for (size_t i = offset; i < segments.size (); i++)
     {
       HIR::PathExprSegment &seg = segments.at (i);
-
-      bool probe_bounds = true;
       bool probe_impls = !reciever_is_generic;
-      bool ignore_mandatory_trait_items = !reciever_is_generic;
 
       // probe the path is done in two parts one where we search impls if no
       // candidate is found then we search extensions from traits
       auto candidates
        = PathProbeType::Probe (prev_segment, seg.get_segment (), probe_impls,
-                               false, ignore_mandatory_trait_items);
+                               false, true /*ignore_mandatory_trait_items*/);
       if (candidates.size () == 0)
        {
          candidates
            = PathProbeType::Probe (prev_segment, seg.get_segment (), false,
-                                   probe_bounds, ignore_mandatory_trait_items);
+                                   true /*probe_bounds*/,
+                                   false /*ignore_mandatory_trait_items*/);
 
          if (candidates.size () == 0)
            {
diff --git a/gcc/testsuite/rust/compile/issue-2070.rs b/gcc/testsuite/rust/compile/issue-2070.rs
new file mode 100644 (file)
index 0000000..46f3520
--- /dev/null
@@ -0,0 +1,22 @@
+trait Foo {
+    fn get(self) -> i32;
+}
+
+struct Bar(i32);
+impl Foo for Bar {
+    fn get(self) -> i32 {
+        self.0
+    }
+}
+
+fn type_bound_test<T: Foo>(a: T) -> i32 {
+    Foo::get(a)
+}
+
+fn main() {
+    let a;
+    a = Bar(456);
+
+    let b;
+    b = type_bound_test(a);
+}