Tidy fix: emplace_back() over for loop

src/include/migraphx/shape.hpp

@@ -65,6 +65,8 @@ struct shape
         std::size_t max = 0;
         std::size_t opt = 1;
 
+        dynamic_dimension() = default;
+
         dynamic_dimension(std::size_t i_min, std::size_t i_max, std::size_t i_opt)
             : min(i_min), max(i_max), opt(i_opt)
         {

src/shape.cpp

@@ -501,10 +501,11 @@ void migraphx_from_value(const value& v, shape& s)
             auto maxes = v.at("max_dyn_dims").to_vector<std::size_t>();
             auto opts  = v.at("opt_dyn_dims").to_vector<std::size_t>();
             assert(mins.size() == maxes.size() and maxes.size() == opts.size());
-            std::vector<shape::dynamic_dimension> dyn_dims;
-            for(int i = 0; i < mins.size(); ++i)
+            auto num_dims = mins.size();
+            std::vector<shape::dynamic_dimension> dyn_dims(num_dims);
+            for(int i = 0; i < num_dims; ++i)
             {
-                dyn_dims.emplace_back(mins[i], maxes[i], opts[i]);
+                dyn_dims.at(i) = shape::dynamic_dimension{mins[i], maxes[i], opts[i]};
             }
             s = shape{migraphx::shape::dyn_data{shape::parse_type(t), dyn_dims}};
         }