In D101691#2731695, @bondhugula wrote:

In D101691#2731306, @eopXD wrote:

Formats fixed, thanks for reviewing.
I don't have the access to commit to llvm, so may you help me to do it?

Done. (I fixed a minor whitespace issue in canonicalize.mlir at line #920 before committing.)

Formats fixed, thanks for reviewing.
I don't have the access to commit to llvm, so may you help me to do it?

Fix format warnings.

I've also added canonicalizeMapAndOperands into SimplifyAffineOp, this will let maps be canonicalized, allowing the maps operations to be folded into the operand.

[mlir][affine] Fix unfolded bounding maps for affine.for

Cherry-pick commit to upstream main.

Hi @bondhugula,
I am trying to rebase with $ git rebase origin/main with the current HEAD on my patching commit.
Then I encountered merge conflicts of file:

@bondhugula I am not able to commit this.
Please help me to commit, thank you.

Yes the comments are all done.
Thank you for the comments.

Adjust testcase to include lower bound composition,
also adjusted testcase to avoid numbered SSA.

[mlir][affine][test] Check if affine.for has bounding maps composed

revert to 1st commit of this bug fix.

This commit shall be added under https://reviews.llvm.org/D99323