anijain2305 commented on issue #4878: [Relay][SimplifyInference] Express 
Softmax as sequence of Relay ops
URL: https://github.com/apache/incubator-tvm/pull/4878#issuecomment-586047123
 
 
   > Great job!
   > I'm not sure whether we want to keep softmax compute & schedule though. If 
someone build training on top of relay ir, they might not enable 
simplifyinference thus need softmax as a single op.
   
   I see. Ok, so I will only change the SimplifyInference pass but keep all the 
compute/schedule etc. Basically, if one disables SimplifyInference, we should 
still be able to run
   

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