zxybazh commented on a change in pull request #7809:
URL: https://github.com/apache/tvm/pull/7809#discussion_r616225480



##########
File path: src/ir/op.cc
##########
@@ -122,6 +124,19 @@ TVM_REGISTER_GLOBAL("ir.RegisterOpAttr")
       }
     });
 
+TVM_REGISTER_GLOBAL("ir.RegisterOpLowerIntrinsic")

Review comment:
       Yeah I have the same concern but I checked both files and found most 
similar functions like `RegisterOpAttr` function in the same file. Therefore, I 
think it better to keep the function here or we would need to migrate all 
related functions.




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


Reply via email to