]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
gccrs: Change assertion of constructor
authorbadumbatish <tanghocle456@gmail.com>
Sun, 21 Jul 2024 22:41:21 +0000 (15:41 -0700)
committerArthur Cohen <arthur.cohen@embecosm.com>
Mon, 17 Mar 2025 15:35:53 +0000 (16:35 +0100)
Change the assert of (expr != nullptr) to
(this->expr != nullptr) because we assigned (std::move(expr)) to
this->expr, no need to assert expr

gcc/rust/ChangeLog:

* ast/rust-expr.h (struct AnonConst):
Change assertion of constructor
(struct InlineAsmOperand):
Change assertion of constructor

gcc/rust/ast/rust-expr.h

index 25afe235e4e83ef59dc8fa515a38f0604797b543..9477bf06d6c0231a45b65a10a703adbcd0c5d357 100644 (file)
@@ -4723,18 +4723,21 @@ struct AnonConst
 {
   NodeId id;
   std::unique_ptr<Expr> expr;
+  AnonConst (NodeId id, std::unique_ptr<Expr> expr)
+    : id (id), expr (std::move (expr))
+  {
+    rust_assert (this->expr != nullptr);
+  }
   AnonConst (const AnonConst &other)
   {
     id = other.id;
-    if (other.expr)
-      expr = other.expr->clone_expr ();
+    expr = other.expr->clone_expr ();
   }
 
   AnonConst operator= (const AnonConst &other)
   {
     id = other.id;
-    if (other.expr)
-      expr = other.expr->clone_expr ();
+    expr = other.expr->clone_expr ();
     return *this;
   }
 };
@@ -4787,7 +4790,7 @@ struct InlineAsmOperand
        std::unique_ptr<Expr> expr)
       : reg (reg), expr (std::move (expr))
     {
-      rust_assert (expr != nullptr);
+      rust_assert (this->expr != nullptr);
     }
 
     In (const struct In &other)
@@ -4816,7 +4819,7 @@ struct InlineAsmOperand
         std::unique_ptr<Expr> expr)
       : reg (reg), late (late), expr (std::move (expr))
     {
-      rust_assert (expr != nullptr);
+      rust_assert (this->expr != nullptr);
     }
 
     Out (const struct Out &other)
@@ -4845,7 +4848,7 @@ struct InlineAsmOperand
           std::unique_ptr<Expr> expr)
       : reg (reg), late (late), expr (std::move (expr))
     {
-      rust_assert (expr != nullptr);
+      rust_assert (this->expr != nullptr);
     }
 
     InOut (const struct InOut &other)
@@ -4877,8 +4880,8 @@ struct InlineAsmOperand
       : reg (reg), late (late), in_expr (std::move (in_expr)),
        out_expr (std::move (out_expr))
     {
-      rust_assert (in_expr != nullptr);
-      rust_assert (out_expr != nullptr);
+      rust_assert (this->in_expr != nullptr);
+      rust_assert (this->out_expr != nullptr);
     }
 
     SplitInOut (const struct SplitInOut &other)
@@ -4911,7 +4914,7 @@ struct InlineAsmOperand
 
     Sym (std::unique_ptr<Expr> expr) : expr (std::move (expr))
     {
-      rust_assert (expr != nullptr);
+      rust_assert (this->expr != nullptr);
     }
     Sym (const struct Sym &other)
     {
@@ -4933,7 +4936,7 @@ struct InlineAsmOperand
     Label (tl::optional<std::string> label_name, std::unique_ptr<Expr> expr)
       : expr (std::move (expr))
     {
-      rust_assert (expr != nullptr);
+      rust_assert (this->expr != nullptr);
       if (label_name.has_value ())
        this->label_name = label_name.value ();
     }