[GitHub] [incubator-tvm] liangfu commented on issue #4694: [VTA] Fix an issue in updating uop_idx in the TensorGemm module

[GitBox <gi...@apache.org>]

liangfu commented on issue #4694: [VTA] Fix an issue in updating uop_idx in the TensorGemm module
URL: https://github.com/apache/incubator-tvm/pull/4694#issuecomment-573539977
 
 
   @kevinyuan Can you help reproduce the results?
   
   A hard requirement is that we should run the module only once with
   ```python
   m.run()
   ```
   , instead of running it in `time_evaluator`.
   
   In addition, I have a patch
   ```c++
   --- a/vta/hardware/dpi/tsim_device.cc
   +++ b/vta/hardware/dpi/tsim_device.cc
   @@ -141,6 +141,8 @@ int VTADPISim() {
          tfp->dump(static_cast<vluint64_t>(trace_count * 2 + 1));
    #endif
        trace_count++;
   +    if ((trace_count % 1000000) == 1)
   +      fprintf(stderr, "[traced %dM cycles]\n", trace_count / 1000000);
        while (top->sim_wait) {
          top->clock = 0;
          std::this_thread::sleep_for(std::chrono::milliseconds(100));
   ```
   to trace the number of cycles in TSIM. 
   It requires 31M cycles to run resnet18_v1 prediction.

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


With regards,
Apache Git Services