manupa-arm commented on a change in pull request #10283:
URL: https://github.com/apache/tvm/pull/10283#discussion_r816683749



##########
File path: include/tvm/runtime/metadata.h
##########
@@ -75,6 +75,13 @@ struct TVMMetadata {
   const struct TVMTensorInfo* outputs;
   /*! \brief Number of elements in `outputs` array. */
   int64_t num_outputs;
+  /*! \brief Memory Pools needed by the AOT run_model function.

Review comment:
       That is right!.
   
   We had run (entry_point), run_model and  __tvm_main__.
   The entry_point is supposed to call run_model, however, run_model is 
identical to tvm_main -- therefore, it was removed due to no need of 
maintaining two symbols for relay and tir versions of main.
   
   So I think it needs to be tvm_main now.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to