Hi @areusch @tqchen @giuseros 

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 ?





---
[Visit 
Topic](https://discuss.tvm.apache.org/t/mini-rfc-name-mangling-in-aot/9907/7) 
to respond.

You are receiving this because you enabled mailing list mode.

To unsubscribe from these emails, [click 
here](https://discuss.tvm.apache.org/email/unsubscribe/65d6ae5991e49e86b42bd4ae8fd7c8ff6f24a8851186a67b8ea165626a1050ec).

Reply via email to