[neurun] Revise OperationValidator for simplicity (#4543)

Revise OperationValidator to override only used methods.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

diff --git a/runtimes/neurun/src/compiler/OperationValidator.cc b/runtimes/neurun/src/compiler/OperationValidator.cc
index 532252e..c734a81 100644 (file)
--- a/runtimes/neurun/src/compiler/OperationValidator.cc
+++ b/runtimes/neurun/src/compiler/OperationValidator.cc
@@ -40,46 +40,6 @@ void OperationValidator::visit(const model::operation::CastNode &node)
   assert(_ctx.at(output_index).shape() == _ctx.at(input_index).shape());
 }
 
-void OperationValidator::visit(const model::operation::Conv2DNode &)
-{
-  // DO NOTHING
-}
-
-void OperationValidator::visit(const model::operation::DepthwiseConv2DNode &)
-{
-  // DO NOTHING
-}
-
-void OperationValidator::visit(const model::operation::MaxPool2DNode &)
-{
-  // DO NOTHING
-}
-
-void OperationValidator::visit(const model::operation::AvgPool2DNode &)
-{
-  // DO NOTHING
-}
-
-void OperationValidator::visit(const model::operation::ConcatNode &)
-{
-  // DO NOTHING
-}
-
-void OperationValidator::visit(const model::operation::FullyConnectedNode &)
-{
-  // DO NOTHING
-}
-
-void OperationValidator::visit(const model::operation::MulNode &)
-{
-  // DO NOTHING
-}
-
-void OperationValidator::visit(const model::operation::ReshapeNode &)
-{
-  // DO NOTHING
-}
-
 void OperationValidator::visit(const model::operation::SoftmaxNode &node)
 {
   VERBOSE(Softmax) << "Configure SOFTMAX operation" << std::endl;
@@ -95,21 +55,6 @@ void OperationValidator::visit(const model::operation::SoftmaxNode &node)
   assert(_ctx.at(scale_index).shape().rank() == 0);
 }
 
-void OperationValidator::visit(const model::operation::StridedSliceNode &)
-{
-  // DO NOTHING
-}
-
-void OperationValidator::visit(const model::operation::TanhNode &)
-{
-  // DO NOTHING
-}
-
-void OperationValidator::visit(const model::operation::LogisticNode &)
-{
-  // DO NOTHING
-}
-
 void OperationValidator::visit(const model::operation::PermuteNode &node)
 {
   VERBOSE(Permute) << "Configure Permute operation" << std::endl;
@@ -123,15 +68,5 @@ void OperationValidator::visit(const model::operation::PermuteNode &node)
   assert(_ctx.at(output_index).shape().rank() == _ctx.at(input_index).shape().rank());
 }
 
-void OperationValidator::visit(const model::operation::AddNode &)
-{
-  // DO NOTHING
-}
-
-void OperationValidator::visit(const model::operation::SubNode &)
-{
-  // DO NOTHING
-}
-
 } // namespace compiler
 } // namespace neurun

diff --git a/runtimes/neurun/src/compiler/OperationValidator.h b/runtimes/neurun/src/compiler/OperationValidator.h
index f4ed533..444d46e 100644 (file)
--- a/runtimes/neurun/src/compiler/OperationValidator.h
+++ b/runtimes/neurun/src/compiler/OperationValidator.h
@@ -41,10 +41,9 @@ public:
   OperationValidator(const neurun::model::operand::Set &ctx) : _ctx{ctx} {}
 
 public:
-#define OP(InternalName, IsNnApi, NnApiName) \
-  virtual void visit(const model::operation::InternalName &) override;
-#include "model/operation/Op.lst"
-#undef OP
+  virtual void visit(const model::operation::CastNode &node) override;
+  virtual void visit(const model::operation::SoftmaxNode &node) override;
+  virtual void visit(const model::operation::PermuteNode &node) override;
 
 private:
   const neurun::model::operand::Set &_ctx;