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 2022/11/05 12:32:36 UTC

[GitHub] [tvm] Lunderberg commented on pull request #13297: [TE] Make `elem_offset` of the buffers created by `te.extern` a variable to avoid crash

Lunderberg commented on PR #13297:
URL: https://github.com/apache/tvm/pull/13297#issuecomment-1304538121

   LGTM, and thank you for making this change!  For context on the fix, this didn't cause an issue previously, because the default `elem_offset=0` could bind to an entire buffer.  It only became necessary when the buffer being bound as an argument to the `te.extern` was a view into a larger backing allocation, which could occur after inlining.


-- 
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