zackcquic commented on pull request #7952:
URL: https://github.com/apache/tvm/pull/7952#issuecomment-830471526


   @tqchen @tkonolige Here is the RFC:
   https://discuss.tvm.apache.org/t/pass-instrument-framework-proposal/9874
   
   And here is some reply for previous questions:
   
   I would like to separate runtime profiling and pass instrumentation, too. 
This PR only focuses on pass profiling mechanisms like the passes time 
profiling, wants to make it easier to add more passes profiling implementations.
   
   You might notice that this PR introduces a new namespace tvm.intrument. It 
intends to cover all instrument related (not limited to pass, but this PR only 
shows pass instrument), instead of mixing instrumentation/profiling code with 
transformation codes in tvm.transform. RuntimeProfiling could be add to this 
namespace, eg: tvm.instrument.Runtime.Profiling.


-- 
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.

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


Reply via email to