leandron commented on a change in pull request #7658:
URL: https://github.com/apache/tvm/pull/7658#discussion_r594625451
##########
File path: python/tvm/driver/tvmc/compiler.py
##########
@@ -196,9 +196,9 @@ def compile_model(
for codegen_from_cli in extra_targets:
codegen =
composite_target.get_codegen_by_target(codegen_from_cli["name"])
partition_function = codegen["pass_pipeline"]
- mod = partition_function(mod, params)
+ mod, codegen_config = partition_function(mod, params)
Review comment:
Fair enough. I thought a bit more about it and as with #7577, we will be
already sending a config dictionary for the partition function. so I think it
is ok to get the same, or a modified version of that dictionary as a result,
which fits with the existing support on TensorRT.
----------------------------------------------------------------
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]
