[Mlir-commits] [mlir] [mlir] Always update ExtractValue to use last container in insert chain (PR #176588)

[llvmlistbot at llvm.org]

neildhar wrote:

> Is the idea to also add some bound for the insert value chain walk?

My thinking is that if #176478 is accepted, it may make sense to only handle the cases that the new `InsertValueOp` canonicalizer will not handle. That should cut down on most of the traversal here. But as long as the traversal is done top-down, it may not matter much in the common case.

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