You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/10/21 19:19:19 UTC

[GitHub] [tvm] csullivan commented on a change in pull request #9342: Support runtime defined function wrapping of library module packed functions

csullivan commented on a change in pull request #9342:
URL: https://github.com/apache/tvm/pull/9342#discussion_r733979316



##########
File path: src/runtime/library_module.h
##########
@@ -65,6 +65,23 @@ class Library : public Object {
   // This is because we do not need dynamic type downcasting.
 };
 
+/*!
+ * \brief Default virtual functor that provides an interface to
+ * wrap a TVMBackendPackedCFunc. Virtual interface allows derivative
+ * runtime's that utilize a library module to to provide custom
+ * function wrapping. By default WrapPackedFunc is used.
+ */
+class PackedFuncWrapper : public Object {
+ public:

Review comment:
       Good suggestion, [updated](https://github.com/apache/tvm/pull/9342/files#diff-c6e5039ebc3f0bccd4c79d58f05c3a7ac51e0719d23477d504c32a0ab1aa2312R81-R86).




-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org