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 2022/09/29 21:31:15 UTC

[GitHub] [tvm] vinx13 opened a new pull request, #12946: [TIR] Remove unused iters from the result of reindex

vinx13 opened a new pull request, #12946:
URL: https://github.com/apache/tvm/pull/12946

   The existence of unused block iters make it difficult for block layout transformation. Previous it relies the flag `simplify_trivial_iter=true` in affine analysis, which is unsafe because simplifying trivial iterators during scheduling stage lose the information of iterators.
   
   cc @spectrometerHBH @junrushao @Hzfengsy 


-- 
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] vinx13 commented on pull request #12946: [TIR] Remove unused iters from the result of reindex

Posted by GitBox <gi...@apache.org>.

vinx13 commented on PR #12946:
URL: https://github.com/apache/tvm/pull/12946#issuecomment-1262857174

   it doesn't because the result is consistent. it only affects im2col results, which is one of the preprocessing step before MultiLevelTiling


-- 
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] junrushao commented on pull request #12946: [TIR] Remove unused iters from the result of reindex

Posted by GitBox <gi...@apache.org>.

junrushao commented on PR #12946:
URL: https://github.com/apache/tvm/pull/12946#issuecomment-1262856011

   I’m not sure though if it’s going to affect tuning or not, because MetaSchedule relies on the structural stability of TIR so that its analysis outcome doesn’t change much. Would you like to elaborate? Thanks a lot!


-- 
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] junrushao merged pull request #12946: [TIR] Remove unused iters from the result of reindex

Posted by GitBox <gi...@apache.org>.

junrushao merged PR #12946:
URL: https://github.com/apache/tvm/pull/12946


-- 
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