]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
gccrs: Prevent used after move on self param
authorPierre-Emmanuel Patry <pierre-emmanuel.patry@embecosm.com>
Fri, 1 Aug 2025 15:31:36 +0000 (17:31 +0200)
committerArthur Cohen <arthur.cohen@embecosm.com>
Tue, 5 Aug 2025 14:37:02 +0000 (16:37 +0200)
gcc/rust/ChangeLog:

* hir/rust-ast-lower-implitem.cc (ASTLowerTraitItem::visit): Remove
use after move.

Signed-off-by: Pierre-Emmanuel Patry <pierre-emmanuel.patry@embecosm.com>
gcc/rust/hir/rust-ast-lower-implitem.cc

index fc9fe1ace4a99b9e6cfea78caeffdc807fed7f26..a35d661338224ed8482a6ed85b7db655a7392888 100644 (file)
@@ -278,6 +278,15 @@ ASTLowerTraitItem::visit (AST::Function &func)
       function_params.push_back (hir_param);
     }
 
+  if (func.has_self_param ())
+    {
+      // insert mappings for self
+      // TODO: Is this correct ? Looks fishy
+      mappings.insert_hir_self_param (&*self_param);
+      mappings.insert_location (self_param->get_mappings ().get_hirid (),
+                               self_param->get_locus ());
+    }
+
   HIR::TraitFunctionDecl decl (func.get_function_name (),
                               std::move (qualifiers),
                               std::move (generic_params),
@@ -301,14 +310,6 @@ ASTLowerTraitItem::visit (AST::Function &func)
     = new HIR::TraitItemFunc (mapping, std::move (decl), std::move (block_expr),
                              func.get_outer_attrs (), func.get_locus ());
   translated = trait_item;
-  if (func.has_self_param ())
-    {
-      // insert mappings for self
-      // TODO: Is this correct ? Looks fishy
-      mappings.insert_hir_self_param (&*self_param);
-      mappings.insert_location (self_param->get_mappings ().get_hirid (),
-                               self_param->get_locus ());
-    }
 
   // add the mappings for the function params at the end
   for (auto &param : trait_item->get_decl ().get_function_params ())