zxybazh commented on PR #13827: URL: https://github.com/apache/tvm/pull/13827#issuecomment-1404352392
Hey thanks for sharing more details. And yes I agree a test for schedule rule as None would be helpful. From a design perspective, we would like to make sure custom schedule rules are applied without interference from other schedule rules. From user perspective, first this provides a default for most of the hardwares like x86 or cuda, and these schedule rules can be customized or replaced if you register a new schedule function into the tvm global registry in the same schedule rule name. Removing those topi default schedule rules wouldnot only make it harder for new users to get started but also impact many current tuning workflows. IMHO, the use case to apply non-custom schedule rules to some topi nn function is actually more of a high-level usage because for the topi nn workloads that you pointed out, usual schedule rules would not be able to extract the best performance comparing to using the schedule rules we specified. On the other hand, these custom schedule rules are target specific, if you do have a good schedule rule set for certain workload, please feel free to send in new custom schedule rule for your hardware, it would benefit even more users :) -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
