You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/04/29 14:21:35 UTC

[GitHub] [incubator-tvm] tqchen edited a comment on pull request #5465: [TIR] Convert if_then_else intrinsics to if-statements

tqchen edited a comment on pull request #5465:
URL: https://github.com/apache/incubator-tvm/pull/5465#issuecomment-621242357


   Thanks @roastduck
   
   Given that it is a quit fundamental  construct that can leads to concise generated code(e.g. ? in CUDA), I think we should keep it throughout the lowering process by default. We can still merge the pass if it is an optional utility
   
   If the goal is mainly to utilize LoopParition, perhaps a better way would be to insert likely into the condition and enable LoopPartition to deal with if_then_else. 
   
   


----------------------------------------------------------------
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.

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