Page MenuHomePhabricator

[MLIR] Fix bug in the method constructing semi affine expression from flattened form
ClosedPublic

Authored by arnab-oss on Nov 3 2022, 11:22 PM.

Details

Summary

Set proper offset to the second element of the index pair when either
lhs or rhs of a local expression is a dimensional identifier, so that
we do not have same index values for more than one local expression.

Diff Detail

Event Timeline

arnab-oss created this revision.Nov 3 2022, 11:22 PM
Herald added a project: Restricted Project. · View Herald Transcript
arnab-oss requested review of this revision.Nov 3 2022, 11:22 PM
bondhugula added inline comments.Nov 3 2022, 11:34 PM
mlir/lib/IR/AffineExpr.cpp
990–992

Please use LLVM/MLIR camel back style names.

arnab-oss updated this revision to Diff 473132.Nov 3 2022, 11:48 PM

Changed variable names to camel back style

springerm accepted this revision.Nov 4 2022, 7:15 AM
This revision is now accepted and ready to land.Nov 4 2022, 7:15 AM
hanchung accepted this revision.Nov 4 2022, 11:10 AM

thanks for the quick fix!