adstraw commented on PR #10910: URL: https://github.com/apache/tvm/pull/10910#issuecomment-1090434886
Note that the "external" constructor for `HexagonBuffer` using `void*` is no longer required after this PR and furthermore `HexagonBuffer` can be simplified as it no longer needs to separately track managed and unmanaged allocations. I think it's OK to follow up with a PR to do this cleanup work in `HexagonBuffer` --- just wanted to note it here. -- 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]
