Thumbs up from me, I'd like to see this proceed to PRs.

- Agree with adding config as a first-class field to IRModule.
- The build API backwards compat should be straightforward by extension of the 
existing `if isinstance` checks. We can emit deprecation warnings for a release 
or two.
- I think you want your tvmc cmdline flags to be interpreted as rewrites on 
CompilerConfig in some uniform way, right?
- We'll need to figure out some verbage for what should go into the config top 
level vs target etc. It's a significant risk the config turns into the union of 
misc fields across targets, passes etc. Many compilers end up in this world by 
accident and it's hard to unwind. But some principles and discipline up front 
should work. 
- Not sure about how to reconcile this with the implicit contexts for Target 
and the PassConfigManager. Personally I'm not a fan of implicit context stacks 
but it is an established TVM patterns.





---
[Visit 
Topic](https://discuss.tvm.apache.org/t/pre-rfc-compilation-configuration-representation/11372/3)
 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/5313b74333aa36277a8518b78d629f2a311babdaa0e187568d1d0166628b6277).

Reply via email to