xqdan commented on issue #4468: [RFC] Data-flow Analysis Functionality on TVM IR
URL: https://github.com/apache/incubator-tvm/issues/4468#issuecomment-563318743
 
 
   @tqchen, what's your suggestion? IMO, low level IR has been there for a 
while, and we've had experience and understanding in low level ir. the post of 
unified ir to me is just a high level proposal, details needs to be discussed 
further,  such as,
   
   The most valuable thing to me is we can make ops white box with unified ir, 
that is to say, we can analyze into the ops, which is totally different with 
the current way, we are using ops as black box in graph framework, or separated 
ir.  with white box ops, we don't need to care about the ops name or the 
formula ops are using, we can optimize them in a general way.  see what's XLA 
doing. 
   
   The abstraction of high level ir matters.  you don't want to lower ops body 
too much from tensor expression into nested loops, since it's a big cost of 
analyzing ir with too much context. 
   
   so I suggest we can keep moving this work on this thread meanwhile we 
discuss how to reuse the solution. 

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to