Lunderberg commented on issue #10505: URL: https://github.com/apache/tvm/issues/10505#issuecomment-1082021039
Also related to `elem_offset`, I think it will need to allow multiple independent offsets. When a tensorized buffer is lowered, the `elem_offset` is used to store the flattened index. For non-flat memory buffers, this requires multiple indices, but only one can be stored there. I think both `elem_offset` and `offset_factor` will require one parameter per physical dimension, and will need to be changed to arrays. I have the proposed changes in #10816 , which should maintain backwards compatibility on the python API, and for constructor calls to `Buffer` on the C++ API. -- 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]
