Lunderberg commented on PR #16250: URL: https://github.com/apache/tvm/pull/16250#issuecomment-1907042466
Taking a look, looks like the the issue is [here](https://github.com/apache/tvm/pull/16250/files#diff-6bacc18d1fa154abbd015f562fc9a73f78249b8eb3d1ef01f4ca1fec50ed1eeeR318). Currently, the `BlockNode::match_buffers` is the point of definition for the view's `data` value, and for the view itself. Based on how it is handled in `LowerMatchBuffer`, it looks like the `shape`, `strides`, and `elem_offset` of the matched buffers should be treated the same as function arguments. That is, they are points of definition if the expression is a variable that hasn't already been defined, but otherwise are points of use. -- 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]
