Re: [PR] [BugFix] Fix Crash Cases Caused by "__tvm_meta__ = None" [tvm]

["slyubomirsky (via GitHub)" <gi...@apache.org>]

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: commits-unsubscribe@tvm.apache.org

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