huajsj commented on a change in pull request #9751:
URL: https://github.com/apache/tvm/pull/9751#discussion_r770185465



##########
File path: src/runtime/pipeline/pipeline_struct.h
##########
@@ -130,42 +145,113 @@ struct OutputMap {
         }
       }
       ICHECK(output_idx >= 0);
-      output_binding_map[output_idx] = binding;
+      output_binding_map_[output_idx] = binding;
     }
   }
 };
+
 /*!
  * \brief The binding or dependency information of each module output 
interface.
  */
-struct PipelineConfig {
-  /*!\brief The key is the module index, this variable records all module 
pipeline configuration
+class ConfigPipelineExecution {
+ private:
+  /*
+   *!\brief The key is the module index, this variable records all module 
pipeline configuration
    * information.
    */
-  std::unordered_map<int, OutputMap> config;
-  OutputMap& operator[](int key) {
-    ICHECK(config.find(key) != config.end());
-    return config[key];
-  }
+  std::unordered_map<int, ConfigOutputBindings> config_;

Review comment:
       fxied.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tvm.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to