programmemory.cpp: avoid some unnecessary copies in `Executor::execut…

…eImpl()` (#6700)
danmar · Aug 16, 2024 · f5af7c6 · f5af7c6
1 parent 601156b
commit f5af7c6
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/programmemory.cpp b/lib/programmemory.cpp
@@ -1529,13 +1529,13 @@ namespace {
                         std::vector<ValueFlow::Value> result =
                             infer(ValueFlow::makeIntegralInferModel(), expr->str(), expr->astOperand1()->values(), {std::move(rhs)});
                         if (!result.empty() && result.front().isKnown())
-                            return result.front();
+                            return std::move(result.front());
                     }
                     if (lhs.isIntValue() && !expr->astOperand2()->values().empty()) {
                         std::vector<ValueFlow::Value> result =
                             infer(ValueFlow::makeIntegralInferModel(), expr->str(), {std::move(lhs)}, expr->astOperand2()->values());
                         if (!result.empty() && result.front().isKnown())
-                            return result.front();
+                            return std::move(result.front());
                     }
                     return unknown();
                 }
@@ -1613,7 +1613,7 @@ namespace {
                             ex.fdepth--;
                             auto r = ex.execute(f->functionScope);
                             if (!r.empty())
-                                result = r.front();
+                                result = std::move(r.front());
                             // TODO: Track values changed by reference
                         }
                     } else {