Adding a TODO for minor techdebt.
The RotationAnalysis builds a dag of operations to use for statically inferring a set of rotation indices given as input to rotation ops. If the computation of those indices involves an insert_slice op, it will likely be incorrect due to this issue. However, it seems unlikely that this will occur, since rotation indices are usually scalar computations.
Adding a TODO for minor techdebt.
The RotationAnalysis builds a dag of operations to use for statically inferring a set of rotation indices given as input to rotation ops. If the computation of those indices involves an insert_slice op, it will likely be incorrect due to this issue. However, it seems unlikely that this will occur, since rotation indices are usually scalar computations.