driazati commented on PR #76:
URL: https://github.com/apache/tvm-rfcs/pull/76#issuecomment-1143757173
Discussion in the community meeting was pretty supportive! Some summary
notes:
* Name prefix option makes sense, change seems like a natural evolution of
MLF
* Before users would have to do the same thing by hand
* Is the deserializer going to be backwards compatible?
* No loader in C++ runtime, only in project API. Eventually MLF scope will
expand to include other things like multiple devices, so C++ loader might be
useful later on. Code typically used along with the same TVM revision so no
need to keep around the old loader. The version number can be used to provide
helpful error messages.
* It would be good to add a description for users on what needs to change,
if anything and reference documentation
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
