comaniac commented on pull request #7656:
URL: https://github.com/apache/tvm/pull/7656#issuecomment-799633968


   > One possible extension would be to try to combine this with the simplify 
reshape above and target patterns like reshape->transpose->reshape or 
transpose->reshape->transpose
   
   That's an interesting point and I just thought about it. However, reshape 
and transpose cannot be equivalent in any cases, because reshape just changes 
the definition of accessing the tensor; while transpose reorganizes the element 
order in memory. Even we combine them to the same patterm, we still need to 
deal with each case separately and result in no code reuse. Please let me know 
if I missed something. Thanks.
   


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


Reply via email to