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/09/04 03:06:39 UTC
[GitHub] [tvm] echuraev edited a comment on pull request #8636: [OpenCL] Add vectorization to cuda conv2d_nhwc schedule
echuraev edited a comment on pull request #8636:
URL: https://github.com/apache/tvm/pull/8636#issuecomment-912897229
> So it appears @mbrookhart has encountered a regression and that regression wasn't reproduced by @echuraev on a slightly different GPU. The possible solution in order to be extra safe would be to introduce a new set of TOPI schedules for vectorized GPU operator implementations. WDYT @jwfromm and @mbrookhart
Sorry, my mistake. I didn't write an update. @mbrookhart has remeasured autotvm tuning on the latest code from my branch and he got the same numbers as he got on the main branch. Now the table can be updated:
| | Results on NVidia 3090 (1024 trials per kernel) | Results on NVidia 3070 Mobile (512 trials per kernel) |
|----------------------------|-------------------------------------------------|-------------------------------------------------------|
| Main (AutoTVM) | 0.17 ms | 0.4504 ms |
| Main (Ansor) | 0.18 ms | 0.4284 ms |
| Code from the PR (AutoTVM) | 0.173 ms | 0.4200 ms |
| Code from the PR (Ansor) | 0.19 ms | 0.4200 ms |
So, we don't have regression on the latest code.
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org