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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
