[Mlir-commits] [mlir] [mlir] Declare promised interfaces for all dialects [2/3] (PR #78368)

Oleksandr Alex Zinenko llvmlistbot at llvm.org
Wed Jan 31 01:27:58 PST 2024


================
@@ -46,6 +49,17 @@ void arith::ArithDialect::initialize() {
       >();
   addInterfaces<ArithInlinerInterface>();
   declarePromisedInterface<ArithDialect, ConvertToLLVMPatternInterface>();
+  declarePromisedInterface<SelectOp,
+                           bufferization::BufferDeallocationOpInterface>();
+  declarePromisedInterface<ConstantOp,
----------------
ftynse wrote:

Cmake doesn't need a dependency to use a header, and it looks like everything needed here is contained in headers because templates.

Whether we _conceptually_ want a dependency here is a good question.

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


More information about the Mlir-commits mailing list