soiferj edited a comment on issue #4939: [Relay][Pass] Sort MergeComposite generated function params alphabetically URL: https://github.com/apache/incubator-tvm/pull/4939#issuecomment-591008923 Sorry, I don't quite understand - where do you think the change should be made? The main issue here is the `FreeVars()` returns free vars in an arbitrary order. I think I see what you're saying - I am a little worried about changing `relay.Function`, since it could potentially impact lots of tests. What do you think?
---------------------------------------------------------------- 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