mbrookhart commented on pull request #7361: URL: https://github.com/apache/tvm/pull/7361#issuecomment-769313450
We could, but we've tested a few models and don't see any performance differences, and @jroesch tells me the purpose of the pass was the first half of plan to do graph-runtime like memory reuse, but the second half was never implemented. Unless we can find a usecase where it helps, I think it makes more sense to disable it entirely until we can get the full feature working. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
