addres review comments.

Author: huajsj

src/runtime/pipeline/pipeline_executor.cc

@@ -127,7 +127,7 @@ void PipelineExecutor::Init(const std::vector<Module>& modules, const std::strin
   // Use JSONReader to load pipeline configuration.
   std::istringstream is(pipeline_json);
   dmlc::JSONReader reader(&is);
-  ConfigPipelineExecution& pipeline_config = this->LoadConfigPipelineExecution(&reader);
+  ConfigPipelineExecution& pipeline_config = this->LoadConfig(&reader);
   ICHECK(!pipeline_config.Empty()) << "The pipeline config information is empty.";
   // Initialize the pipeline function class used for pipeline thread pool management
   // and schedule etc. This function returns the number of output.

src/runtime/pipeline/pipeline_executor.h

@@ -131,7 +131,7 @@ class TVM_DLL PipelineExecutor : public ModuleNode {
   /*!\brief How many outputs are in this pipeline executor.*/
   size_t num_outputs_ = 0;
   /*!\brief Json loader.*/
-  ConfigPipelineExecution& LoadConfigPipelineExecution(dmlc::JSONReader* reader) {
+  ConfigPipelineExecution& LoadConfig(dmlc::JSONReader* reader) {
     reader->BeginObject();
     std::string key;
     while (reader->NextObjectItem(&key)) {

src/runtime/pipeline/pipeline_struct.h

@@ -27,14 +27,13 @@
 #include <unordered_map>
 #include <utility>
 #include <vector>
-#define GLOBAL_MODULE_INDEX -1
 /*!
  * \brief All binding information of a output interface.
  */
 class ConfigBindings {
  public:
   /*!\brief Whether this binding is bound to the PipelineExecutor output interface.*/
-  bool IsGlobalOutput() const { return global_output_index_ > GLOBAL_MODULE_INDEX; }
+  bool IsGlobalOutput() const { return global_output_index_ > -1; }
 
   /*!
    * \brief Create a module interface map from JSONReader.