zhiics commented on pull request #5770:
URL: https://github.com/apache/incubator-tvm/pull/5770#issuecomment-645177764


   @FrozenGene The current implementation doesn't actually change any 
user/frontend APIs. We added the `MetadataModule` to take care of weight 
initialization and separate it from code compilation. So you would have:
   
   ```
   MetadataModule:
       - DSO module
       - Accelerator CSourceModule0
       - Accelerator CSourceModule1
   ```
   
   As you can see, `MetadataModule` manages the params, but we will still have 
code in the submodules. The serialization and deserialization make sure that 
the module structure is always maintained correctly. Therefore, we can still 
have params/lib/json in one module. 
   
   In other words, it should not affect the way we package the final artifacts, 
but it changes the way we compile certain modules (e.g. CSourceModule) and the 
way we initialize modules that need constant data.
   
   


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