This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Use interfaces in MathToLibm
ClosedPublic

Authored by tpopp on Jun 1 2021, 4:04 AM.

Details

Summary

Previously, this assumed use of ModuleOp and FuncOp. There is no need to
restrict this, and using interfaces allows these patterns to be used
during dialect conversion to LLVM.

Some assertions were removed due to inconsistent implementation of
FunctionLikeOps.

Diff Detail

Event Timeline

tpopp created this revision.Jun 1 2021, 4:04 AM
tpopp requested review of this revision.Jun 1 2021, 4:04 AM
ftynse accepted this revision.Jun 1 2021, 4:13 AM
ftynse added inline comments.
mlir/lib/Conversion/MathToLibm/MathToLibm.cpp
91–92

Nit: drop llvm::

This revision is now accepted and ready to land.Jun 1 2021, 4:13 AM
This revision was automatically updated to reflect the committed changes.
tpopp marked an inline comment as done.Jun 1 2021, 4:58 AM