I went through the new proposal and the PR. This looks much better to me from 
the perspecitive of functionality.

One concern in my mind is the long term maintaince. It seems like we will have 
more and more new features dealing with a set of tasks. As @tqchen mentioned in 
another [RFC](https://github.com/apache/incubator-tvm/issues/4188), it might be 
better to make a task pass manager to manage such processes, but we should be 
able to integrate this one to the task pass manager later on once we have it 
ready.

@eqy @kevinthesun do you guys have any other concerns?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-tvm/issues/4150#issuecomment-553030890

Reply via email to