Move makeValueBinaryOpFromLocal to ValueExpr.*

src/optimization/Combiner.cpp

@@ -1127,12 +1127,6 @@ InstructionWalker optimizations::combineArithmeticOperations(
     return it;
 }
 
-std::shared_ptr<ValueExpr> makeValueBinaryOpFromLocal(Value& left, ValueBinaryOp::BinaryOp binOp, Value& right)
-{
-    return std::make_shared<ValueBinaryOp>(
-        std::make_shared<ValueTerm>(left), binOp, std::make_shared<ValueTerm>(right));
-}
-
 // try to convert shl to mul and return it as ValueExpr
 std::shared_ptr<ValueExpr> shlToMul(Value& value, const intermediate::Operation* op)
 {

src/optimization/ValueExpr.cpp

@@ -148,6 +148,12 @@ std::string ValueBinaryOp::to_string() const
     return "(" + left->to_string() + " " + opStr + " " + right->to_string() + ")";
 }
 
+std::shared_ptr<ValueExpr> optimizations::makeValueBinaryOpFromLocal(Value& left, ValueBinaryOp::BinaryOp binOp, Value& right)
+{
+    return std::make_shared<ValueBinaryOp>(
+            std::make_shared<ValueTerm>(left), binOp, std::make_shared<ValueTerm>(right));
+}
+
 bool ValueTerm::operator==(const ValueExpr& other) const
 {
     if(auto otherTerm = dynamic_cast<const ValueTerm*>(&other))

src/optimization/ValueExpr.h

@@ -72,6 +72,8 @@ namespace vc4c
             std::shared_ptr<ValueExpr> right;
         };
 
+        std::shared_ptr<ValueExpr> makeValueBinaryOpFromLocal(Value& left, ValueBinaryOp::BinaryOp binOp, Value& right);
+
         class ValueTerm : public ValueExpr
         {
         public: