[Mlir-commits] [mlir] [mlir] Change `tensor.extract/insert` to take static/dynamic indices. (PR #104488)

[Mehdi Amini]

https://github.com/joker-eph requested changes to this pull request.

What is the motivation for this?

In general I see this as an anti-pattern to have operation accept both an attribute and an SSA value.
I thought the "experiment" with the few ops that adopted this form in the past was deemed like an accepted "bad idea" in the end?

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