Uh oh!
There was an error while loading. Please reload this page.
[WIP] Revert lower tvm builtin change - #8643
Conversation
d449808 to
4cf4dc5Comparejunrushao
commented
Aug 10, 2021
I’m not actually an expert in vta or opencl. CC @tmoreau89 if you are interested :-) |
tmoreau89
commented
Aug 10, 2021
Thanks for the WIP PR, please let me know when it's ready for review @zhanghaohit - and also please provide a short summary of the PR in the description box, thanks! |
tmoreau89
commented
Aug 12, 2021
@zhanghaohit looks like the CIs are green! Let me know when you'd like to change this PR to not WIP |
zhanghaohit
commented
Aug 13, 2021
Hi @tmoreau89, this PR is actually the same as #8274 proposed by @mbrookhart . It is to show how to pass the CI if we want to revert the lower_tvm_builtin_change. I think it is better to let @mbrookhart apply the changes to his PR #8274, which has already much discussion and context there. I'll close this PR first, in order not to confuse. |
mbrookhart
commented
Aug 13, 2021
Huh, I thought I did this, to the tutorial, but it doesn't seem to be in my PR! Oops! I'll do this there, thanks. |
mbrookhart
commented
Aug 13, 2021
😳 Oops. Apparently I made this change locally and never pushed it upstream. Thank you for double checking me. |
mbrookhart
commented
Aug 13, 2021
Also, sorry I missed this last week, I was out on PTO. |
tmoreau89
commented
Aug 13, 2021
Ah no problem, my reaction times have been slow lately, I'm glad we're making forward progress. |
No description provided.