kumasento commented on issue #4748: [RELAY] Support RelayBuild with Only 
Constants
URL: https://github.com/apache/incubator-tvm/issues/4748#issuecomment-583730264
 
 
   Hi @zhiics @tqchen I'm interested in this issue and would be happy to help.
   
   From a user's perspective, we would normally expect that `relay.build` 
returns an empty `lib`. An empty `lib` can be `NULL`/`None`, but it may cause 
confusion like this issue. As @zhiics mentioned, using assertion may raise 
unexpected issues for end users as well.
   
   One workaround for this problem, which I'm currently implementing, is 
inserting a dummy function (with empty content) into the `lowered_funcs` map 
before passing to module building. In this way, we won't break other parts of 
the codebase, and the returned `lib` still makes sense.
   
   If you prefer this way, I can continue implementing it and create a PR later.
   
   Thanks

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


With regards,
Apache Git Services

Reply via email to