slyubomirsky commented on PR #16634:
URL: https://github.com/apache/tvm/pull/16634#issuecomment-1973891325

   I've figured out, thanks to @Lunderberg's comments in my PR, that the 
failure in 
`tests/python/tir-transform/test_tir_transform_force_narrow_index_to_i32.py::test_thread_axis2`
 is due to line 354 in `scr/script/ir_builder/tir/ir.cc`. If you change 
   ```c++
       IterVar iter_var(Range(nullptr), Var("iter", dtype), 
IterVarType::kThreadIndex, thread);
   ```
   to
   ```c++
       IterVar iter_var(Range(nullptr), Var("iter", DataType::Int(32)), 
IterVarType::kThreadIndex, thread);
   ```
   (what it was before the merge commit that Eric pointed out), the test will 
pass. However, I'm not sure we want to fix the data type like that.


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