This is an archive of the discontinued LLVM Phabricator instance.

[MLIR] Give AffineStoreOp and AffineLoadOp Memory SideEffects.
ClosedPublic

Authored by tpopp on Apr 27 2020, 7:55 AM.

Details

Summary

This change results in tests also being changed to prevent dead
affine.load operations from being folded away during rewrites.

Also move AffineStoreOp and AffineLoadOp to an ODS file.

Diff Detail

Event Timeline

tpopp created this revision.Apr 27 2020, 7:55 AM
mehdi_amini accepted this revision.Apr 27 2020, 12:55 PM
This revision is now accepted and ready to land.Apr 27 2020, 12:55 PM
rriddle accepted this revision.Apr 27 2020, 1:11 PM
rriddle added inline comments.
mlir/include/mlir/Dialect/Affine/IR/AffineOps.td
688

Can you keep this file sorted alphabetically?

tpopp updated this revision to Diff 260569.Apr 28 2020, 2:04 AM
tpopp marked an inline comment as done.

Reorder definition of ops and handle a rebase.

nicolasvasilache accepted this revision.Apr 28 2020, 6:32 AM
This revision was automatically updated to reflect the committed changes.