Merge pull request #519 from diffblue/trans-trace-assignmentt

`trans_tracet::assignmentt` now has full constructor
diffblue · May 22, 2024 · 1679d2b · 1679d2b
2 parents 44a1b82 + 3cdaa9e
commit 1679d2b
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 19 deletions.
diff --git a/src/trans-netlist/trans_trace.h b/src/trans-netlist/trans_trace.h
@@ -28,8 +28,14 @@ class trans_tracet
       exprt lhs;
       exprt rhs;
       source_locationt location;
-
-      assignmentt():location(source_locationt::nil())
+
+      assignmentt(
+        exprt __lhs,
+        exprt __rhs,
+        source_locationt __location = source_locationt::nil())
+        : lhs(std::move(__lhs)),
+          rhs(std::move(__rhs)),
+          location(std::move(__location))
       {
       }
     };

diff --git a/src/trans-netlist/trans_trace_netlist.cpp b/src/trans-netlist/trans_trace_netlist.cpp
@@ -144,15 +144,9 @@ trans_tracet compute_trans_trace(
 
         value=ch+value;
       }
-
-      state.assignments.push_back(trans_tracet::statet::assignmentt());
 
-      trans_tracet::statet::assignmentt &assignment=
-        state.assignments.back();
-
-      assignment.lhs=symbol.symbol_expr();
-      assignment.rhs=bitstring_to_expr(value, var.type);
-      assignment.location.make_nil();
+      state.assignments.emplace_back(
+        symbol.symbol_expr(), bitstring_to_expr(value, var.type));
     }
 
     // check the property

diff --git a/src/trans-word-level/trans_trace_word_level.cpp b/src/trans-word-level/trans_trace_word_level.cpp
@@ -62,10 +62,7 @@ trans_tracet compute_trans_trace(
         {
           if(symbol.value.is_constant())
           {
-            trans_tracet::statet::assignmentt assignment;
-            assignment.rhs = symbol.value;
-            assignment.lhs = symbol.symbol_expr();
-            state.assignments.push_back(assignment);
+            state.assignments.emplace_back(symbol.symbol_expr(), symbol.value);
           }
         }
         else
@@ -79,11 +76,8 @@ trans_tracet compute_trans_trace(
           if(value_expr == indexed_symbol_expr)
             value_expr = nil_exprt();
 
-          trans_tracet::statet::assignmentt assignment;
-          assignment.rhs.swap(value_expr);
-          assignment.lhs = symbol.symbol_expr();
-
-          state.assignments.push_back(assignment);
+          state.assignments.emplace_back(
+            symbol.symbol_expr(), std::move(value_expr));
         }
       }
     }