From: 박종현/동작제어Lab(SR)/Staff Engineer/삼성전자 <jh1302.park@samsung.com>
Date: Mon, 8 Oct 2018 07:33:51 +0000 (+0900)
Subject: [coco] Require Instr on Step construction (#1779)
X-Git-Tag: nncc_backup~1604
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4e31b627757121ca589642bb89fb6fa071c715f3;p=platform%2Fcore%2Fml%2Fnnfw.git

[coco] Require Instr on Step construction (#1779)

This commit disables default constructor of Step, and explicitly requires
Instr on construction.

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

diff --git a/contrib/coco/core/include/coco/IR/Step.h b/contrib/coco/core/include/coco/IR/Step.h
index 24d7a2d..31dad43 100644
--- a/contrib/coco/core/include/coco/IR/Step.h
+++ b/contrib/coco/core/include/coco/IR/Step.h
@@ -29,7 +29,10 @@ namespace coco
 class Step final
 {
 public:
-  Step() = default;
+  explicit Step(Instr *instr) : _instr{instr}
+  {
+    // DO NOTHING
+  }
 
 public:
   ~Step() { op(nullptr); }
@@ -40,7 +43,6 @@ public:
 
 public:
   Instr *instr(void) const { return _instr; }
-  void instr(Instr *i) { _instr = i; }
 
 private:
   Op *_op = nullptr;
diff --git a/contrib/coco/core/include/coco/IR/UnitF.h b/contrib/coco/core/include/coco/IR/UnitF.h
index 27a8798..66e3c78 100644
--- a/contrib/coco/core/include/coco/IR/UnitF.h
+++ b/contrib/coco/core/include/coco/IR/UnitF.h
@@ -32,7 +32,10 @@ namespace coco
 class UnitF final : public FeatureInstr
 {
 public:
-  UnitF() { _step.instr(this); }
+  UnitF() : _step{this}
+  {
+    // DO NOTHING
+  }
 
 public:
   UnitF *asUnitF(void) override { return this; }
diff --git a/contrib/coco/core/src/IR/Eval.cpp b/contrib/coco/core/src/IR/Eval.cpp
index f6a51df..23b7f9f 100644
--- a/contrib/coco/core/src/IR/Eval.cpp
+++ b/contrib/coco/core/src/IR/Eval.cpp
@@ -34,7 +34,7 @@ std::set<coco::Bag *> &operator+=(std::set<coco::Bag *> &res, const coco::Object
 namespace coco
 {
 
-Eval::Eval() : _out{this}
+Eval::Eval() : _out{this}, _step{this}
 {
   // DO NOTHING
 }