Lunderberg commented on PR #16674: URL: https://github.com/apache/tvm/pull/16674#issuecomment-1979827243
Ah, thank you, and that makes sense. That would also explain why the increased footprint isn’t seen when using `LazyTransformParams`, since the tuple doesn’t exist as a parameter. If I understand correctly, we could achieve a similar effect by unpacking the entire tuple on its first use, though that could be less robust to later lowering passes. Since we have the in-place updates, it makes sense to use them, and I like the change. -- 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]
