remove required method

apache · Mar 9, 2019 · a0863b4 · a0863b4
1 parent ee359fb
commit a0863b4
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 52 deletions.
diff --git a/include/tvm/relay/pass.h b/include/tvm/relay/pass.h
@@ -144,11 +144,6 @@ class PassNode : public RelayNode {
    */
   virtual void SetContext(const PassContext& pass_ctx) = 0;
 
-  /*!
-   * \brief Get the required passes for this pass as a vector of std::string.
-   */
-  virtual std::vector<std::string> Required() const = 0;
-
   /*!
    * \brief Execute the optimization pass using a functor.
    *

diff --git a/src/relay/pass/pass_manager.cc b/src/relay/pass/pass_manager.cc
@@ -47,9 +47,6 @@ class ModulePassNode : public PassNode {
    */
   Module operator()(const Module& mod) const final;
 
-  /*! \brief Collect the required passes for this module pass. */
-  std::vector<std::string> Required() const final;
-
   /*!
    * \brief Get the pass information/meta data.
    */
@@ -116,9 +113,6 @@ class FunctionPassNode : public PassNode {
    */
   Module operator()(const Module& mod) const final;
 
-  /*! \brief Collect the required passes for this module pass. */
-  std::vector<std::string> Required() const final;
-
   /*!
    * \brief Get the pass information/meta data.
    */
@@ -217,8 +211,6 @@ class SequentialPassNode : public PassNode {
    */
   void ResolveDependency(const Module& mod);
 
-  std::vector<std::string> Required() const final;
-
   TVM_DLL std::vector<std::string> DisabledPasses() const;
 
   /*!
@@ -289,19 +281,6 @@ Module ModulePassNode::operator()(const Module& mod) const {
   return updated_mod;
 }
 
-std::vector<std::string> ModulePassNode::Required() const {
-  std::vector<std::string> required;
-  PassInfo pass_info = GetPassInfo();
-  CHECK(pass_info.defined())
-      << "Cannot get the required passes from undefined PassInfo object."
-      << "\n";
-  for (const auto& it : pass_info.operator->()->required) {
-    const auto* str = it.as<tvm::ir::StringImm>();
-    required.push_back(str->value);
-  }
-  return required;
-}
-
 void ModulePassNode::SetContext(const PassContext& pass_ctx) {
   pass_ctx_ = pass_ctx;
 }
@@ -352,19 +331,6 @@ bool FunctionPassNode::SkipFunction(const Function& func) const {
   return pval && pval->value != 0;
 }
 
-std::vector<std::string> FunctionPassNode::Required() const {
-  std::vector<std::string> required;
-  PassInfo pass_info = GetPassInfo();
-  CHECK(pass_info.defined())
-      << "Cannot get the required passes from undefined PassInfo object."
-      << "\n";
-  for (const auto& it : pass_info.operator->()->required) {
-    const auto* str = it.as<tvm::ir::StringImm>();
-    required.push_back(str->value);
-  }
-  return required;
-}
-
 SequentialPass SequentialPassNode::make(PassInfo pass_info,
                                         tvm::Array<Pass> passes,
                                         tvm::Array<tvm::Expr> disabled) {
@@ -388,19 +354,6 @@ Module SequentialPassNode::operator()(const Module& module) const {
   return mod;
 }
 
-std::vector<std::string> SequentialPassNode::Required() const {
-  std::vector<std::string> required;
-  PassInfo pass_info = GetPassInfo();
-  CHECK(pass_info.defined())
-      << "Cannot get the required passes from undefined PassInfo object."
-      << "\n";
-  for (const auto& it : pass_info.operator->()->required) {
-    const auto* str = it.as<tvm::ir::StringImm>();
-    required.push_back(str->value);
-  }
-  return required;
-}
-
 void SequentialPassNode::ResolveDependency(const Module& mod) {
   // TODO(zhiics) Implement it.
   // 1. Consider the required passes for each pass.