tqchen opened a new pull request #7918:
URL: https://github.com/apache/tvm/pull/7918


   Previouslyw we set the context of the DSO module to be the root module.
   This can cause problem when the root module is not the dso module and
   get destructued early (but we still need the dso module).
   
   This PR makes the following change to fix the problem.
   
   - Merge multiple DSO modules to one during export.
   - Set the context to be the (only one) dso module.
   - Updated testcase to cover the problem.
   
   The enhancement creates some restrictions on the dso import hierachy
   (all dso modules needs to be merged without a cycle). This is the case
   for our current use scenario. The merged logic is also more consistent
   as the library itself is merged.
   


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