fix review comments

src/onnx/onnx.cpp

@@ -206,6 +206,16 @@ struct onnx_parser
         return out_lens;
     }
 
+    instruction_ref make_contiguous(instruction_ref ins)
+    {
+        if (ins->get_shape().standard())
+        {
+            return ins;
+        }
+
+        return prog.add_instruction(op::contiguous{}, ins);
+    }
+
     template <class T>
     instruction_ref add_broadcastable_binary_op(instruction_ref arg0, instruction_ref arg1, T x)
     {
@@ -437,11 +447,7 @@ struct onnx_parser
             s.visit([&](auto v) { copy(v, std::back_inserter(op.dims)); });
         }
 
-        if(!args[0]->get_shape().standard())
-        {
-            args[0] = prog.add_instruction(op::contiguous{}, args[0]);
-        }
-
+        args[0] = make_contiguous(args[0]);
         return prog.add_instruction(op, args[0]);
     }
 
@@ -490,15 +496,8 @@ struct onnx_parser
         {
             axis = parse_value(attributes.at("axis")).at<int>();
         }
-
-        if(!args[0]->get_shape().standard())
-        {
-            args[0] = prog.add_instruction(op::contiguous{}, args[0]);
-        }
-        if(!args[1]->get_shape().standard())
-        {
-            args[1] = prog.add_instruction(op::contiguous{}, args[1]);
-        }
+        args[0] = make_contiguous(args[0]);
+        args[1] = make_contiguous(args[1]);
 
         op::gather op{axis};
         return prog.add_instruction(op, std::move(args));