[Mlir-commits] [mlir] Reorder OpBuildGen.cpp (PR #129976)

[llvmlistbot at llvm.org]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: None (wenzhi-cui)

<details>
<summary>Changes</summary>

verifyOp will erase the op so must be called later.

---
Full diff: https://github.com/llvm/llvm-project/pull/129976.diff


1 Files Affected:

- (modified) mlir/unittests/TableGen/OpBuildGen.cpp (+1-1) 


``````````diff
diff --git a/mlir/unittests/TableGen/OpBuildGen.cpp b/mlir/unittests/TableGen/OpBuildGen.cpp
index b4a5c316d63d6..ec1d5f63f8c24 100644
--- a/mlir/unittests/TableGen/OpBuildGen.cpp
+++ b/mlir/unittests/TableGen/OpBuildGen.cpp
@@ -303,8 +303,8 @@ TEST_F(OpBuildGenTest, BuildMethodsInherentDiscardableAttrs) {
   // place works.
   auto op7b = builder.create<test::TableGenBuildOp7>(loc, TypeRange{},
                                                      ValueRange{}, attrs);
-  verifyOp(op7b, {}, {}, attrs);
   ASSERT_EQ(op7b.getProperties().getAttr0(), attrs[0].getValue());
+  verifyOp(op7b, {}, {}, attrs);
 }
 
 } // namespace mlir

``````````

</details>


https://github.com/llvm/llvm-project/pull/129976