zhiics commented on issue #4939: [Relay][Pass] Sort MergeComposite generated 
function params alphabetically
URL: https://github.com/apache/incubator-tvm/pull/4939#issuecomment-591032603
 
 
   IMHO, this might seem like a workaround, but I think it doesn't really solve 
the ordering problem of pattern matching because we can still have different 
orders for the args if they are not vars. For example,  `add(add(a, b), 
sub(c,d))` is equivalent to `add(sub(c,d), add(a, b))`, but we cannot match it 
without providing the exactly needed pattern. The mismatch in var is just one 
of the simplified version for us.
   

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to