FrozenGene edited a comment on pull request #5753:
URL: https://github.com/apache/incubator-tvm/pull/5753#issuecomment-658014208


   > @FrozenGene : With the latest change, i am not able to find dealing with 
multiple modules like previously done. Can you please point me to one or give 
an example how to do it. Thanks!
   
   @ANSHUMAN87  Ah...yes. Latest change of this pr doesn't contain this part. 
The main reasons  is our compiler doesn't be ready for it. For example, imagine 
we have one resnet18 model and resnet50 model for CPU, our compiler will not 
generate unique name for them. Both models will have the same function name 
like `fused_nn_contrib_conv2d_NCHWc_add`. So when our compiler is ready for 
this, we could enable multi model support. Current pr has considered this 
situation and we could add this support easily. Previous pr you see the multi 
model support is one model for CPU, one model for GPU, it just work around and 
bypass this issue.


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