[GitHub] [tvm] kevinthesun commented on pull request #7361: Remove MemoryPlan from VM passes

2021-01-28 Thread GitBox
kevinthesun commented on pull request #7361: URL: https://github.com/apache/tvm/pull/7361#issuecomment-769487569 Thanks @mbrookhart @masahi @icemelon9 @zhiics This is an automated message from the Apache Git Service. To

[GitHub] [tvm] kevinthesun commented on pull request #7361: Remove MemoryPlan from VM passes

2021-01-28 Thread GitBox
kevinthesun commented on pull request #7361: URL: https://github.com/apache/tvm/pull/7361#issuecomment-769487309 LGTM This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [tvm] kevinthesun commented on pull request #7361: Remove MemoryPlan from VM passes

2021-01-28 Thread GitBox
kevinthesun commented on pull request #7361: URL: https://github.com/apache/tvm/pull/7361#issuecomment-769469662 @mbrookhart Sure. You can refer to [tf ssd integration test](https://github.com/apache/tvm/blob/main/tests/python/frontend/tensorflow/test_forward.py#L3090).

[GitHub] [tvm] kevinthesun commented on pull request #7361: Remove MemoryPlan from VM passes

2021-01-28 Thread GitBox
kevinthesun commented on pull request #7361: URL: https://github.com/apache/tvm/pull/7361#issuecomment-769412338 @mbrookhart Thanks. It would be great if you can try tf ssd and fasterrcnn so that we can ensure there is no regression for tf models as well.

[GitHub] [tvm] kevinthesun commented on pull request #7361: Remove MemoryPlan from VM passes

2021-01-28 Thread GitBox
kevinthesun commented on pull request #7361: URL: https://github.com/apache/tvm/pull/7361#issuecomment-769296390 cc @zhiics @icemelon9 This is an automated message from the Apache Git Service. To respond to the message,