I think its best to use “tvm” prefix nonetheless. – so we dont pollute a namespace based on a user given variable.
I dont follow why a “prefix” necessarily mean user being able to select it? If “prefix” is not the right term we should not call it a prefix. The goal of this RFC is to propose mechanism to reduce namespace pollution. Therefore having “tvm” states that this belongs tvm codegen’d artifacts and moreover allowing a model name allows to further categorize artifacts of multiple compilations.
Therefore, I’d suggest we use :
- tvm_MYNET_run (lets drop the func
)
and (if a model name is not given)
- tvm_run
@tqchen , I’d assume if a model name is not given the second option would be backward compatible DSOModule loader ?
(edited to reflect @mjs suggestions)