MargaretQian commented on a change in pull request #10722:
URL: https://github.com/apache/tvm/pull/10722#discussion_r837779910
##########
File path: python/tvm/driver/tvmc/compiler.py
##########
@@ -243,7 +244,8 @@ def compile_model(
PassContext.
additional_target_options: Optional[Dict[str, Dict[str, Any]]]
Additional target options in a dictionary to combine with initial
Target arguments
-
+ use_vm: bool
Review comment:
Discussed offline -- since "vm" is not already registered as an
Executor, it would require:
1) registering the vm as an executor here:
https://github.com/apache/tvm/blob/main/src/relay/backend/executor.cc#L96
2) fitting it into the codegen pattern here:
https://github.com/apache/tvm/blob/main/src/relay/backend/build_module.cc#L158-L168
decided that this will fall outside of the scope of this PR
--
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]