[GitHub] [tvm] AndrewZhaoLuo opened a new pull request, #11618: [AutoTVM][Autoscheduler] Default build funcs inherit PassContext

[GitBox <gi...@apache.org>]

AndrewZhaoLuo opened a new pull request, #11618:
URL: https://github.com/apache/tvm/pull/11618

   These funcs used building during the tuning process now inherit PassContext so we can inject things like pass instrumentation into the build process during tuning.
   
   Note, due to the fact subprocesses are used for building by default, we can't naively do something like
   
   ```
   with PassContext(...):
      tune()
   ```
   
   But what does allow us to do is wrap the existing build functions with new PassContexts and call those instead.


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] AndrewZhaoLuo closed pull request #11618: [AutoTVM][Autoscheduler] Default build funcs inherit PassContext

[GitBox <gi...@apache.org>]

AndrewZhaoLuo closed pull request #11618: [AutoTVM][Autoscheduler] Default build funcs inherit PassContext
URL: https://github.com/apache/tvm/pull/11618


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] areusch commented on pull request #11618: [AutoTVM][Autoscheduler] Default build funcs inherit PassContext

[GitBox <gi...@apache.org>]

areusch commented on PR #11618:
URL: https://github.com/apache/tvm/pull/11618#issuecomment-1149285156

   should we do something like assert if the PassContext changes between Tuner construction and calling `tune()`? would help reduce the confusion, i think.


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] AndrewZhaoLuo commented on pull request #11618: [AutoTVM][Autoscheduler] Default build funcs inherit PassContext

[GitBox <gi...@apache.org>]

AndrewZhaoLuo commented on PR #11618:
URL: https://github.com/apache/tvm/pull/11618#issuecomment-1150416201

   Just noticed I accidently opened this on main repo instead of my own fork, moved to https://github.com/apache/tvm/pull/11632


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] AndrewZhaoLuo commented on pull request #11618: [AutoTVM][Autoscheduler] Default build funcs inherit PassContext

[GitBox <gi...@apache.org>]

AndrewZhaoLuo commented on PR #11618:
URL: https://github.com/apache/tvm/pull/11618#issuecomment-1150161800

   Hmm, I don't think that would be super helpful since the issue comes from the fact that during tuning, the PassContext is ignored by the children processes which actually build stuff during tuning.
   
   I guess we can make it so PassContext is inherited instead, which I guess I'll take a look, it will make things even more transparent.


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org