elvin-n opened a new pull request #8707:
URL: https://github.com/apache/tvm/pull/8707


   Previously knob values depended on order of explicit cfg update and 
cfg.define_split calls in fallback mode
   
   I.e. schedule depend where and when default KNOB values are defined. I.e. 
for [dense 
x8](https://github.com/apache/tvm/blob/main/python/tvm/topi/x86/dense.py#L168)6 
we define a split space and then if we are in fallback mode, we manually point 
some heuristic factors. And everything works well. But for batch_matmul x86 we 
define certain values in case of fallback in [compute func 
first](https://github.com/apache/tvm/blob/main/python/tvm/topi/x86/batch_matmul.py#L74)
 and only then [in 
schedule](https://github.com/apache/tvm/blob/main/python/tvm/topi/x86/batch_matmul.py#L120)
 these values will be overwritten by first pair from search space that is 
(-1,1).
   
   This fix makes any order of execution reuse predefined values in fallback 
mode


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


Reply via email to