tqchen commented on PR #104:
URL: https://github.com/apache/tvm-rfcs/pull/104#issuecomment-1845530627
Just to circle back here a bit. the main root issue is that we are using
runtime::DataType, which is supposely being concrete through out the TIR node.
This places restrictions on what we can normally represent. A more
comprehensive update would change the PrimExpr's field to also an object, as
per StructInfo in the relax. That would requires bit more thinking, which
likely can get around the issues mentioned in the thread(of passing around
runtime::DataType which is not an object).
I think in short term making the protocol of lanes = -1 and lanes = -8(for
vscale(8)) may not be a bad idea. The main reason is I cannot think of another
possible use of the lanes field other than for the SVE.
--
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]
