masahi edited a comment on pull request #7233:
URL: https://github.com/apache/tvm/pull/7233#issuecomment-757032109
> Autotvm does this for external libraries which are all extern, so it will
work here
@tkonolige I like the idea of separating sorting based implementation of
scatter, so I want to try this. Can you point me where in the codebase autotvm
deals with external libs? I found something like
`@autotvm.register_topi_compute("conv2d_cudnn.cuda")` but I'm not sure how it
interacts with normal conv2d ops.
One issue is that currently sorting based approach is only implemented for
1D scatter. For higher dimensions, I think sorting based approach is a bad
idea. So dispatching decision needs to take input dimension into account (not
sure if this could be a problem for autotvm or relay strategy).
----------------------------------------------------------------
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]
