ANSHUMAN87 edited a comment on pull request #5922:
URL: https://github.com/apache/incubator-tvm/pull/5922#issuecomment-649639015


   @tqchen, @yzhliu : I have one concern here(However not related to this PR).
   As you see the expr below:
   expr = (((x0 - floordiv((0 - (x1*5)), 2)) - 1) + floordiv((37 + ((x1 + 
7)*-5)), 2))
   
   should be evaluated to actually
   expr = x0
   
   but with current simplify stage(rewrite_simplify + canonical_simplify), it 
evaluates to below:
   expr = ((floordiv((0 - (x1: int32*5)), 2) + x0: int32) - floordiv((x1*-5), 
2))
   
   But when i run stages as (rewrite_simplify + canonical_simplify + 
rewrite_simplify) it evaluates perfectly to x0.
   
   So may be we need to run these stages with some customized config every time 
or until there are no more changes?


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


Reply via email to