CallahanRL opened a new issue #8616:
URL: https://github.com/apache/tvm/issues/8616


   
https://github.com/apache/tvm/blob/9f29e2a94923604aeff88d1a8f3efaf0c3bd0655/src/arith/rewrite_simplify.cc#L857
   
   I have a counter example to this rewrite. It takes this code 
   `floormod((((threadIdx.x*50) + (b.i.fused.j.fused.inner.outer.inner*8)) + 
b.i.fused.j.fused.inner.inner.s_1), 20)`
   to this code:
   `ramp((floormod(((threadIdx.x*50) + 
(b.i.fused.j.fused.inner.outer.inner*8)), 20)*32), 32, 8)`
   
   The code determines the  modular set to have base 0 and coeff 2 but when it 
computes ramp_max for lanes=8, it incorrectly determines that 
ramp_min==ramp_max. Perhaps this is always a problem with ceoff*lanes <  c2val?
   
   I can't provide a test case because it requires a modified version of 
vectorize_loop.cc which I can't yet share.
   
   cc @jcf94 @tqchen 


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to