This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Add EqualOp and NotEqualOp to complex dialect.
ClosedPublic

Authored by akuegel on May 20 2021, 3:56 AM.

Details

Reviewers
pifon2a
herhut
Summary

For complex numbers, only equal and not equal comparisons are defined.
So this patch adds those comparisons as two different ops, instead of going the
way with one op and an attribute.

Diff Detail

Event Timeline

akuegel created this revision.May 20 2021, 3:56 AM
akuegel requested review of this revision.May 20 2021, 3:56 AM
pifon2a accepted this revision.May 20 2021, 4:04 AM
pifon2a added inline comments.
mlir/include/mlir/Dialect/Complex/IR/ComplexOps.td
174

nit: can we call them EqualOp and NotEqualOp like in TOSA and TF dialects?

This revision is now accepted and ready to land.May 20 2021, 4:04 AM
akuegel updated this revision to Diff 346695.May 20 2021, 4:13 AM

Rename to EqualOp and NotEqualOp.

akuegel retitled this revision from [mlir] Add IsEqualOp and IsNotEqualOp to complex dialect. to [mlir] Add EqualOp and NotEqualOp to complex dialect..May 20 2021, 4:13 AM
akuegel added inline comments.
mlir/include/mlir/Dialect/Complex/IR/ComplexOps.td
174

Sure, makes sense :)
Done.