You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/02/22 11:34:38 UTC

[GitHub] [tvm] FrozenGene commented on pull request #7495: [Relay VM] VM Profiler Supports Profiling Instruction Stats

FrozenGene commented on pull request #7495:
URL: https://github.com/apache/tvm/pull/7495#issuecomment-783310773


   Thanks @monklof Personally, I don't like we have to turn on one option and rebuild TVM if we want to profile (this situation is the same as  DebugGraphRuntime).  Like TRT, we have function called as `reportLayerTime` to do it but we don't have to use one version like Debug TRT. (see https://github.com/NVIDIA/TensorRT/blob/master/samples/opensource/sampleNMT/sampleNMT.cpp#L1127). 
   
   Of course, we don't have to create one class named as Profiler in TRT to do this. But whether we could simplify our usage of profiling? I think we could consider this VM and GraphRuntime together (Do we need DebugGraphRuntime and have to rebuild TVM when we want to profile?). cc @comaniac @tqchen @jcf94 want to listen more voices. 


----------------------------------------------------------------
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:
users@infra.apache.org