masahi commented on a change in pull request #10724:
URL: https://github.com/apache/tvm/pull/10724#discussion_r833951559
##########
File path: python/tvm/topi/arm_cpu/conv2d_alter_op.py
##########
@@ -101,9 +102,6 @@ def _alter_conv2d_layout(attrs, inputs, tinfos, out_type):
# we then assume it's not necessary to alter this op.
return None
cfg = dispatch_ctx.query(target, workload)
- if cfg.is_fallback: # if is fallback, clear query cache and return None
- autotvm.task.clear_fallback_cache(target, workload)
- return None
Review comment:
The CI has passed. I found that Giuseppe's im2col based conv2d
implementation can fail to tensorize in the fallback mode, so I partially
restored the fallback return path above.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
