[loco] Clean up deprecated CanonicalNodeImpl (#6903)
author박종현/On-Device Lab(SR)/Staff Engineer/삼성전자 <jh1302.park@samsung.com>
Mon, 26 Aug 2019 07:33:42 +0000 (16:33 +0900)
committerGitHub Enterprise <noreply-CODE@samsung.com>
Mon, 26 Aug 2019 07:33:42 +0000 (16:33 +0900)
Let's clean up CanonicalNodeImpl deprecated by CanonicalNodeDef.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>
compiler/loco/include/loco/IR/CanonicalNodeDecl.h
compiler/loco/src/IR/Graph.test.cpp

index a226d51..872edbb 100644 (file)
@@ -36,14 +36,6 @@ struct CanonicalNode : public Node
   template <typename T> T accept(CanonicalNodeMutableVisitorBase<T> *);
 };
 
-template <CanonicalOpcode Code> struct CanonicalNodeImpl : public CanonicalNode
-{
-  virtual ~CanonicalNodeImpl() = default;
-
-  uint32_t opnum(void) const final { return static_cast<uint32_t>(Code); }
-  CanonicalOpcode opcode(void) const final { return Code; }
-};
-
 template <CanonicalOpcode Code, template <typename T> class... Mixins>
 struct CanonicalNodeDef : public virtual CanonicalNode, public Mixins<CanonicalNode>...
 {
index 0641238..0b77e34 100644 (file)
@@ -108,7 +108,7 @@ namespace
 {
 // temp node with multple params for ctor. loco::CanonicalOpcode::ReLU is used for simplicity
 class ParamCtorNode
-    : public loco::FixedArityNode<0, loco::CanonicalNodeImpl<loco::CanonicalOpcode::ReLU>>
+    : public loco::CanonicalNodeDef<loco::CanonicalOpcode::ReLU, loco::FixedArity<0>::Mixin>
 {
 public:
   ParamCtorNode(int i, float f)