zxybazh commented on a change in pull request #7809:
URL: https://github.com/apache/tvm/pull/7809#discussion_r617093473
##########
File path: python/tvm/ir/op.py
##########
@@ -115,3 +127,43 @@ def _register(v):
return v
return _register(value) if value is not None else _register
+
+
+def register_op_intrin_lowering(
+ op_name,
+ target,
+ f=None,
+ level=10,
+ override=False,
+):
Review comment:
Thanks for pointing that out. I thought about this issue again and I
think here `f` is used overwhelmingly more than `level` and `override`. In most
cases, it's just `op_name`, `target` and `f`. Therefore, I insist that we keep
the current order for most intuitive function usage.
--
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:
[email protected]
