electriclilies commented on pull request #7086: URL: https://github.com/apache/tvm/pull/7086#issuecomment-744635938
@u99127 I think this change won't remove any nodes from the relay IR, since reverse_reshape just constructs a reshape node with the reverse attribute set to true. There is no standalone reverse_reshape op, only a constructor. This makes transforming a graph with a reshape in it is difficult because when you encounter a reshape op, you must check the value of reverse in its attributes, then copy all of the attributes except for reverse to a new set of attributes, then pass those attributes into the reshape constructor or the reverse_reshape constructor depending on what the value of the reverse attribute is. ---------------------------------------------------------------- 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]
