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/12/22 12:41:12 UTC

[GitHub] [tvm] lhutton1 opened a new pull request #9793: [microNPU] Re-enable LayoutOptimizer pass

lhutton1 opened a new pull request #9793:
URL: https://github.com/apache/tvm/pull/9793


   It looks like in #9597 the LayoutOptimizer pass was accidentally removed. Probably due to a race condition when merging PR's. Re-enabling this feature.
   
   cc @ekalda @manupa-arm @mbaret
   


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [tvm] manupa-arm merged pull request #9793: [microNPU] Re-enable LayoutOptimizer pass

Posted by GitBox <gi...@apache.org>.
manupa-arm merged pull request #9793:
URL: https://github.com/apache/tvm/pull/9793


   


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [tvm] manupa-arm commented on pull request #9793: [microNPU] Re-enable LayoutOptimizer pass

Posted by GitBox <gi...@apache.org>.
manupa-arm commented on pull request #9793:
URL: https://github.com/apache/tvm/pull/9793#issuecomment-1000208505


   Thanks all! Lets add a test to make sure it doesn't get removed silently...


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [tvm] manupa-arm commented on pull request #9793: [microNPU] Re-enable LayoutOptimizer pass

Posted by GitBox <gi...@apache.org>.
manupa-arm commented on pull request #9793:
URL: https://github.com/apache/tvm/pull/9793#issuecomment-1000207233


   Oops.... I also think we need to have at least a single integration test that requires the this pass being run -- maybe in a follow up?


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [tvm] lhutton1 commented on pull request #9793: [microNPU] Re-enable LayoutOptimizer pass

Posted by GitBox <gi...@apache.org>.
lhutton1 commented on pull request #9793:
URL: https://github.com/apache/tvm/pull/9793#issuecomment-999566063


   Yes I think that would make more sense!


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [tvm] lhutton1 commented on pull request #9793: [microNPU] Re-enable LayoutOptimizer pass

Posted by GitBox <gi...@apache.org>.
lhutton1 commented on pull request #9793:
URL: https://github.com/apache/tvm/pull/9793#issuecomment-1000209169


   Yes that's sensible, will do!


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org