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