Skip to content

Commit ab77a70

Browse files
AviadCoAviad Cohen
and
Aviad Cohen
authored
Fixed wrong check OperationEquivalenceTest.HashWorksWithFlags (#142210)
The check was meant to check `IgnoreProperties` works as expected but operated on the wrong operation. Co-authored-by: Aviad Cohen <aviad.cohen2@mobileye.com>
1 parent 0996bfc commit ab77a70

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

mlir/unittests/IR/OperationSupportTest.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -350,10 +350,10 @@ TEST(OperationEquivalenceTest, HashWorksWithFlags) {
350350
auto req2 = b.getI32IntegerAttr(60);
351351
Operation *opWithProperty2 = b.create<test::OpAttrMatch1>(
352352
b.getUnknownLoc(), req2, nullptr, nullptr, req2);
353-
EXPECT_NE(getHash(op1, OperationEquivalence::None),
354-
getHash(op2, OperationEquivalence::None));
355353
EXPECT_EQ(getHash(opWithProperty1, OperationEquivalence::IgnoreProperties),
356354
getHash(opWithProperty2, OperationEquivalence::IgnoreProperties));
355+
EXPECT_NE(getHash(opWithProperty1, OperationEquivalence::None),
356+
getHash(opWithProperty2, OperationEquivalence::None));
357357
opWithProperty1->destroy();
358358
opWithProperty2->destroy();
359359
}

0 commit comments

Comments
 (0)