This is an archive of the discontinued LLVM Phabricator instance.

[mlir][pdl] Some ops are missing `NoSideEffect`
ClosedPublic

Authored by Mogball on Jan 20 2022, 12:14 PM.

Details

Summary

Querying or building constraints on types, operands, results, and attributes are side-effect free in both the matcher and rewriter. The ops should be marked as such.

Diff Detail

Event Timeline

Mogball created this revision.Jan 20 2022, 12:14 PM
Mogball requested review of this revision.Jan 20 2022, 12:14 PM
rriddle accepted this revision.Jan 20 2022, 12:16 PM
This revision is now accepted and ready to land.Jan 20 2022, 12:16 PM
This revision was landed with ongoing or failed builds.Jan 20 2022, 12:21 PM
This revision was automatically updated to reflect the committed changes.