[Mlir-commits] [mlir] [mlir][scf] Uplift `scf.while` to `scf.for` (PR #76108)

Ivan Butygin llvmlistbot at llvm.org
Thu Dec 21 09:30:11 PST 2023


Hardcode84 wrote:

I'm not against making it a part of canonicalization and it will work for our case, but I think there is still big chance that it can break downstream workflows.

So, if we are really want to make it part of canonicalization I suggest to split it into 2 parts:
* Introduce it as separate pass first (this PR)
* Promote to canonicalization with separate PR and forum PSA

So, in case we need to revert it, we can revert only the canonicalizer part

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


More information about the Mlir-commits mailing list