Pass SizeVector by const reference in ie_layout.hpp (#965)

diff --git a/inference-engine/include/ie_layouts.h b/inference-engine/include/ie_layouts.h
index 7a5dbd4..6da977b 100644 (file)
--- a/inference-engine/include/ie_layouts.h
+++ b/inference-engine/include/ie_layouts.h
@@ -56,7 +56,7 @@ public:
      * @param offset offset to the current memory block
      * @param dimOffsets per-dimension offset from the padding to actual data,
      */
-    BlockingDesc(const SizeVector& blocked_dims, const SizeVector& order, size_t offset, SizeVector dimOffsets);
+    BlockingDesc(const SizeVector& blocked_dims, const SizeVector& order, size_t offset, const SizeVector& dimOffsets);
     /**
      * @brief The constructor allows to create blocking descriptors for blocked memory
      *
@@ -66,8 +66,8 @@ public:
      * @param dimOffsets per-dimension offset from the padding to actual data,
      * @param strides strides for each dimension
      */
-    BlockingDesc(const SizeVector& blocked_dims, const SizeVector& order, size_t offset, SizeVector dimOffsets,
-                 SizeVector strides);
+    BlockingDesc(const SizeVector& blocked_dims, const SizeVector& order, size_t offset,
+                 const SizeVector& dimOffsets, const SizeVector& strides);
 
     /**
      * @brief Returns the blocked dimensions vector
@@ -159,7 +159,7 @@ public:
      * @param dims memory dimensions
      * @param blockDesc blocking descriptor
      */
-    TensorDesc(const Precision& precision, SizeVector dims, const BlockingDesc& blockDesc);
+    TensorDesc(const Precision& precision, const SizeVector& dims, const BlockingDesc& blockDesc);
     /**
      * @brief The constructor creates the tensor descriptor using standard layout
      *
@@ -167,7 +167,7 @@ public:
      * @param dims memory dimensions
      * @param layout memory layout
      */
-    TensorDesc(const Precision& precision, SizeVector dims, Layout layout);
+    TensorDesc(const Precision& precision, const SizeVector& dims, Layout layout);
     /**
      * @brief The constructor creates the empty tensor descriptor with precision and layout
      *
@@ -297,7 +297,7 @@ public:
      * @param dims the vector of dimensions
      * @return the standard memory layout
      */
-    static Layout getLayoutByDims(SizeVector dims);
+    static Layout getLayoutByDims(const SizeVector& dims);
 
 private:
     /**

diff --git a/inference-engine/src/inference_engine/ie_layouts.cpp b/inference-engine/src/inference_engine/ie_layouts.cpp
index 1a2f9f6..a35b547 100644 (file)
--- a/inference-engine/src/inference_engine/ie_layouts.cpp
+++ b/inference-engine/src/inference_engine/ie_layouts.cpp
@@ -9,7 +9,7 @@
 
 using namespace InferenceEngine;
 
-TensorDesc::TensorDesc(const Precision& precision, SizeVector dims, Layout layout)
+TensorDesc::TensorDesc(const Precision& precision, const SizeVector& dims, Layout layout)
     : precision(precision), blockingDesc(dims, layout) {
     this->dims = dims;
     this->layout = layout;
@@ -19,7 +19,7 @@ TensorDesc::TensorDesc(const Precision& precision, Layout layout): precision(pre
     this->layout = layout;
 }
 
-TensorDesc::TensorDesc(const Precision& precision, SizeVector dims, const BlockingDesc& blockDesc)
+TensorDesc::TensorDesc(const Precision& precision, const SizeVector& dims, const BlockingDesc& blockDesc)
     : dims(dims), precision(precision), blockingDesc(blockDesc) {
     if (dims.size() == 0 || blockingDesc.getBlockDims().size() == 0) {
         layout = Layout::SCALAR;
@@ -158,7 +158,7 @@ bool TensorDesc::operator!=(const TensorDesc& rhs) const {
     return !(*this == rhs);
 }
 
-Layout TensorDesc::getLayoutByDims(SizeVector dims) {
+Layout TensorDesc::getLayoutByDims(const SizeVector& dims) {
     switch (dims.size()) {
     case 0:
         return Layout::SCALAR;
@@ -249,7 +249,7 @@ BlockingDesc::BlockingDesc(const SizeVector& blocked_dims, const SizeVector& ord
 }
 
 BlockingDesc::BlockingDesc(const SizeVector& blocked_dims, const SizeVector& order, size_t offset,
-                           SizeVector dimOffsets)
+                           const SizeVector& dimOffsets)
     : BlockingDesc(blocked_dims, order) {
     this->offsetPadding = offset;
     if (blocked_dims.size() != dimOffsets.size())
@@ -258,7 +258,7 @@ BlockingDesc::BlockingDesc(const SizeVector& blocked_dims, const SizeVector& ord
 }
 
 BlockingDesc::BlockingDesc(const SizeVector& blocked_dims, const SizeVector& order, size_t offset,
-                           SizeVector dimOffsets, SizeVector strides)
+                           const SizeVector& dimOffsets, const SizeVector& strides)
     : BlockingDesc(blocked_dims, order) {
     this->offsetPadding = offset;
     if (blocked_dims.size() != strides.size()) THROW_IE_EXCEPTION << "Strides are not initialized for all dimensions.";