[Mlir-commits] [mlir] Remove Pure attribute from Linalg::IndexOp. (PR #68894)

[llvmlistbot at llvm.org]

MaheshRavishankar wrote:

I don't see why `linalg.index` should not be pure. It has no side-effecting semantics. It should be CSEd. The issue where you shouldn't CSE this is with nested Linalg ops which AFAICS is illegal udage of Linalg ops.

If someone can summarize/maybe start a discourse post we can get convergence there instead of a PR

https://github.com/llvm/llvm-project/pull/68894