You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/01/28 19:18:46 UTC

[GitHub] [tvm] mbrookhart edited a comment on pull request #7361: Remove MemoryPlan from VM passes

mbrookhart edited a comment on pull request #7361:
URL: https://github.com/apache/tvm/pull/7361#issuecomment-769313450


   We could, but we've tested a few onnx and pytorch 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:
users@infra.apache.org