zxybazh commented on code in PR #12342:
URL: https://github.com/apache/tvm/pull/12342#discussion_r940909594


##########
tests/python/unittest/test_meta_schedule_tune_tir.py:
##########
@@ -66,6 +66,7 @@ def test_tune_matmul_cpu():
                 max_trials_global=32,
             ),
             work_dir=work_dir,
+            blocks=["update"],

Review Comment:
   Thanks for adding the test and it works in this case. Furthermore, we are 
assuming there's an AutoInline schedule rule which would inline block 'A' if 
available so that we can tell if `tune_block` api works. We could simplify and 
avoid this assumption by directly supplying it with a schedule rule to inline 
the block it gets so that it will trigger an error if blocks api is not 
correctly called (because we cannot inline both blocks right).



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