quic-sanirudh edited a comment on pull request #6138: URL: https://github.com/apache/incubator-tvm/pull/6138#issuecomment-671139917
Hi @tqchen Thanks for the comments. I was aware of the issue with rfactor primitive, and so I added a check that fails if `init` is used with the rfactor primitive. We could do something similar for the crossthread_allreduce as well. My thought process was that since we expect the user to understand what custom initialization means, they would in general be cautious of using it when running parallel reductions. Also, I'm not aware of the TIR level scheduling that's planned, is there an RFC or PR where this is being discussed that I can read about. 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: us...@infra.apache.org