valueflow.cpp: avoid unnecessary copies in valueFlowFunctionReturn()

Author: firewave

lib/valueflow.cpp

@@ -5960,10 +5960,10 @@ static void valueFlowFunctionReturn(TokenList& tokenlist, ErrorLogger& errorLogg
 
         bool hasKnownValue = false;
 
-        for (const ValueFlow::Value& v : getCommonValuesFromTokens(returns)) {
-            setFunctionReturnValue(function, tok, v, settings, false);
+        for (ValueFlow::Value& v : getCommonValuesFromTokens(returns)) {
             if (v.isKnown())
                 hasKnownValue = true;
+            setFunctionReturnValue(function, tok, std::move(v), settings, false);
         }
 
         if (hasKnownValue)