[Docs] Add tvm.s_tir.tensor_intrin API reference and remove empty legacy tvm/tir directory#19386
[Docs] Add tvm.s_tir.tensor_intrin API reference and remove empty legacy tvm/tir directory#19386MasterJH5574 merged 3 commits intoapache:mainfrom
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces documentation for the tvm.s_tir.tensor_intrin module and updates type hints across the arith package, transitioning from tvm.tir.Var to tvm.tirx.Var. The review feedback highlights a typo in the relax_map documentation within bound.py and a factual error in the neg_inf docstring in int_set.py where it was incorrectly described as representing positive infinity.
| Domain of variables used to help deduction. | ||
|
|
||
| relax_map : Map[tvm.tir.Var, IntSet] | ||
| relax_map : Map[tvm.tirx.Var, IntSet] |
There was a problem hiding this comment.
There is a typo in the description for relax_map on the following line: 'fomain' should be 'domain'. Since this block is being updated, it's a good opportunity to fix it.
| Returns | ||
| ---------- | ||
| neg_inf : tvm.tir.Var | ||
| neg_inf : tvm.tirx.Var |
There was a problem hiding this comment.
The docstring for neg_inf incorrectly describes the return value as indicating 'positive infinity' (both in the summary at line 178 and the parameter description at line 183). It should be 'negative infinity'.
2 participants
as per title