Lunderberg commented on PR #16938: URL: https://github.com/apache/tvm/pull/16938#issuecomment-2081169184
> We don;t need to worry about TE, since we will be able to support buffer binding side. I brought up TE as a potential place to implement a change, but don't think it is the best option. While buffer-binding is supported in the `tvm.build` API, it is not supported in `relax.BlockBuilder.call_te`, nor in the `te.create_prim_func` used by `BlockBuilder.call_te`. Because it isn't exposed by these APIs, an implementation at this level would need to change the defaults all the way down to TE, which is a wider-reaching change than I'd like to make. > In some cases it is indeed useful to introduce views, in some of those cases we might be able to inline views directly into the previous operation, such that the buffer themselfs still comes with zero offset. Agreed, and I like this as the long-term direction. -- 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]
