Hzfengsy opened a new pull request #8855:
URL: https://github.com/apache/tvm/pull/8855
In this PR, I have fixed two bugs:
1. Incorrectly detect access region for opaque access in
`PlanAndUpdateBufferAllocationLocation`
Currently, the pass `PlanAndUpdateBufferAllocationLocation` will
re-calculate the access region. It will enlarge the access region even though
with manually specified access regions. Actually, the only thing we should do
is erase the buffers allocated inside the block.
2. Incorrectly analyze the buffer access region created by `match_buffer`.
`block_access_region_detector` will only detector buffers in the given
map. As for the buffer created by `match_buffer`, we SHOULD only consider the
buffers whose source buffer is in the given map. But we detect all buffers
created by `match_buffer` by mistake. Also, the previous testcase is incorrect,
so we can pass it even the logic is wrong.
Thanks @MasterJH5574 for reporting this error.
--
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]
