[Mlir-commits] [mlir] [mlir][affine][gpu] support unroll dynamic value and add gpu unroll test. (PR #128113)

[lonely eagle]

linuxlonelyeagle wrote:

Maybe you are worried whether this is a safe enough method. However, in my opinion, it is indeed a safe enough method.
The previous implementation replaced the operands with their minimum and maximum values.But I found a potential problem, if there are multiple operands in affineMap, the result may not be accurate. But ValueBoundInterface can get the maximum and minimum values ​​of the whole map result, I think it should be safe and correct enough.

One more thing is, my program runs just fine using this pass.
So I think the changes in this PR are a safe and reliable enough solution.

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