Page MenuHomePhabricator

Update normalizeAffineFor to canonicalize maps/operands before using them
ClosedPublic

Authored by bondhugula on Mar 7 2022, 12:06 AM.

Details

Summary

Update normalizeAffineFor utility to canonicalize maps and operands
before using them.

Diff Detail

Event Timeline

bondhugula created this revision.Mar 7 2022, 12:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 7 2022, 12:06 AM
bondhugula requested review of this revision.Mar 7 2022, 12:06 AM
ftynse accepted this revision.Mar 7 2022, 2:36 AM
ftynse added inline comments.
mlir/test/Dialect/Affine/affine-loop-normalize.mlir
31

We don't care that the for is in the next line after the func declaration, so just CHECK should suffice here.

This revision is now accepted and ready to land.Mar 7 2022, 2:36 AM
bondhugula updated this revision to Diff 413437.Mar 7 2022, 5:19 AM

CHECK instead of CHECK-NEXT.

bondhugula marked an inline comment as done.Mar 7 2022, 5:19 AM
bondhugula added inline comments.
mlir/test/Dialect/Affine/affine-loop-normalize.mlir
31

Sure.

This revision was landed with ongoing or failed builds.Mar 7 2022, 5:30 AM
This revision was automatically updated to reflect the committed changes.
bondhugula marked an inline comment as done.