When I read this RFC, I was confused because I read that the compilation flow 
**currently** goes from 
> TF/PyTorch/ONNX -> Relay -> TE (based on TE schedule) -> TIR -> C++/CUDA

And then I read:
> TensorIR is a brand new low-level IR with full scheduling support. Here are 
> some [...]

And then see references to `TIR` throughout the rest of the RFC, which seem 
unclear to me if these references are to the **new** `TensorIR`, or the **old** 
`TIR`.

Can we clarify both here and in the code base moving forward whether we are 
referring to `TensorIR` or the old `TIR`? I think there are two ways of doing 
this going forward and we should explicitly pick one:

1. Have `TIR` refer to the old version of TIR, and always use `TensorIR` when 
talking about the new IR.
2. Be clear that TensorIR moving forward will be replacing the old `TIR`, so 
that any reference to `TIR` could be referring to the new or old version (and 
should be clarified if not obvious from context).

Someone more knowledgeable than me should pick one of those (or correct me 
and/or point out other options if I've gotten anything wrong here).





---
[Visit 
Topic](https://discuss.tvm.apache.org/t/rfc-tensorir-a-schedulable-ir-for-tvm/7872/39)
 to respond.

You are receiving this because you enabled mailing list mode.

To unsubscribe from these emails, [click 
here](https://discuss.tvm.apache.org/email/unsubscribe/793aaee1938382b9b1215945f8f86517563aa3bbf8683fc0f33e1b09fb523744).

Reply via email to