[Mlir-commits] [llvm] [mlir] [MLIR] convert OpAsmDialectInterface using ODS (PR #171488)

[Jacques Pienaar]

================
@@ -11,24 +11,18 @@ def NoDefaultMethod : DialectInterface<"NoDefaultMethod"> {
 
   let methods = [
     InterfaceMethod<
-      /*desc=*/        "Check if it's an example dialect",
-      /*returnType=*/  "bool",
-      /*methodName=*/  "isExampleDialect",
-      /*args=*/        (ins)
-      >,
-      InterfaceMethod<
-      /*desc=*/        "second method to check if multiple methods supported",
-      /*returnType=*/  "unsigned",
-      /*methodName=*/  "supportSecondMethod",
-      /*args=*/        (ins "::mlir::Type":$type)
+      "Check if it's an example dialect", "bool", "isExampleDialect", (ins)
+    >,
+    InterfaceMethod<
+      "second method to check if multiple methods supported",
+      "unsigned", "supportSecondMethod", (ins "::mlir::Type":$type)
       >
-
   ];
 }
 
 // DECL:   class NoDefaultMethod : public {{.*}}DialectInterface::Base<NoDefaultMethod>
 // DECL:   public:
-// DECL-NEXT:   NoDefaultMethod(::mlir::Dialect *dialect) : Base(dialect) {}
+/// DECL:   NoDefaultMethod(::mlir::Dialect *dialect) : Base(dialect) {}
----------------
jpienaar wrote:

```suggestion
// DECL:   NoDefaultMethod(::mlir::Dialect *dialect) : Base(dialect) {}
```

To match the others.

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