[Mlir-commits] [mlir] [mlir][ODS] Fix TableGen for AttrOrTypeDef::hasStorageCustomConstructor (PR #147957)

[Mehdi Amini]

================
@@ -668,10 +668,10 @@ void DefGen::emitHashKey() {
 }
 
 void DefGen::emitConstruct() {
-  Method *construct = storageCls->addMethod<Method::Inline>(
+  Method *construct = storageCls->addMethod(
       strfmt("{0} *", def.getStorageClassName()), "construct",
       def.hasStorageCustomConstructor() ? Method::StaticDeclaration
-                                        : Method::Static,
+                                        : Method::StaticInline,
----------------
joker-eph wrote:

Can we add a check in `addMethod` that detects the incompatibility between `Method::Inline` and `StaticDeclaration`?

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