This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Remove the PatternState class and simplify PatternMatchResult.
ClosedPublic

Authored by rriddle on Mar 15 2020, 2:16 PM.

Details

Summary

PatternState was a mechanism to pass state between the match and rewrite calls of a RewritePattern. With the rise of matchAndRewrite, this class is unused and unnecessary. This revision removes PatternState and simplifies PatternMatchResult to just be a LogicalResult. A future revision will replace all usages of PatternMatchResult/matchSuccess/matchFailure with LogicalResult equivalents.

Diff Detail

Event Timeline

rriddle created this revision.Mar 15 2020, 2:16 PM
flaub added a subscriber: flaub.Mar 15 2020, 8:41 PM
jpienaar accepted this revision.Mar 16 2020, 11:32 AM

LG given it is unused.

This revision is now accepted and ready to land.Mar 16 2020, 11:32 AM
This revision was automatically updated to reflect the committed changes.