This is an archive of the discontinued LLVM Phabricator instance.

[MLIR][Presburger] MaybeLocalRepr: add explicit bool() for convenience
ClosedPublic

Authored by arjunp on Feb 2 2022, 7:12 AM.

Details

Diff Detail

Event Timeline

arjunp created this revision.Feb 2 2022, 7:12 AM
arjunp requested review of this revision.Feb 2 2022, 7:12 AM
bondhugula added inline comments.
mlir/lib/Analysis/Presburger/IntegerPolyhedron.cpp
875–889

Is this the only use?

arjunp added inline comments.Feb 2 2022, 8:08 AM
mlir/lib/Analysis/Presburger/IntegerPolyhedron.cpp
875–889

These are the only ones I'm aware of currently. There will be another in PresburgerSet in a subsequent patch.

Groverkss accepted this revision.Feb 5 2022, 3:42 AM

LGTM. I think this will be useful for some changes I have in mind also.

This revision is now accepted and ready to land.Feb 5 2022, 3:42 AM