[coco] Introduce create method for Generic FeatureLayout (#1731)

For consistency, this commit introduces create method for Generic
FeatureLayout.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

diff --git a/contrib/coco/core/include/coco/IR/FeatureLayouts.h b/contrib/coco/core/include/coco/IR/FeatureLayouts.h
index d5cb869..8244d25 100644 (file)
--- a/contrib/coco/core/include/coco/IR/FeatureLayouts.h
+++ b/contrib/coco/core/include/coco/IR/FeatureLayouts.h
@@ -90,7 +90,7 @@ public:
  */
 class Generic final : public FeatureLayout
 {
-public:
+private:
   Generic(const FeatureShape &shape);
 
 public:
@@ -114,6 +114,9 @@ private:
 
 private:
   std::vector<ElemID> _content;
+
+public:
+  static std::unique_ptr<Generic> create(const FeatureShape &shape);
 };
 
 } // namespace FeatureLayouts

diff --git a/contrib/coco/core/src/IR/FeatureLayouts.cpp b/contrib/coco/core/src/IR/FeatureLayouts.cpp
index 325dcf5..ffe066d 100644 (file)
--- a/contrib/coco/core/src/IR/FeatureLayouts.cpp
+++ b/contrib/coco/core/src/IR/FeatureLayouts.cpp
@@ -156,5 +156,11 @@ void Generic::reorder(const nncc::core::ADT::feature::Layout &l)
   }
 }
 
+std::unique_ptr<Generic> Generic::create(const nncc::core::ADT::feature::Shape &shape)
+{
+  // NOTE It is impossible to use make_unique here as the constructor is private
+  return std::unique_ptr<Generic>{new Generic{shape}};
+}
+
 } // namespace FeatureLayouts
 } // namespace coco

diff --git a/contrib/coco/core/src/IR/FeatureObject.cpp b/contrib/coco/core/src/IR/FeatureObject.cpp
index ca1bf5e..b7f2270 100644 (file)
--- a/contrib/coco/core/src/IR/FeatureObject.cpp
+++ b/contrib/coco/core/src/IR/FeatureObject.cpp
@@ -29,7 +29,7 @@ namespace coco
 
 FeatureObject::FeatureObject(const nncc::core::ADT::feature::Shape &shape)
 {
-  _layout = make_unique<FeatureLayouts::Generic>(shape);
+  _layout = FeatureLayouts::Generic::create(shape);
 }
 
 FeatureObject::~FeatureObject()