Lunderberg commented on PR #14488: URL: https://github.com/apache/tvm/pull/14488#issuecomment-1496508010
From some quick tests, either representation could be removed without much difficulty. If we were getting rid of one of the two representations, I'd lean toward keeping the `PointerType(...)` version, since it is also needed to express a non-global scope. That said, having a distinction between a pointer to a value and an opaque handle is probably a good one to keep around, so I'd lean toward @junrushao's suggestion. -- 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]
