[mlir] Allow for using interface class name in ODS interface definitions
authorAlex Zinenko <zinenko@google.com>
Sun, 15 Nov 2020 16:49:37 +0000 (17:49 +0100)
committerAlex Zinenko <zinenko@google.com>
Tue, 17 Nov 2020 13:28:55 +0000 (14:28 +0100)
commit88f25bda1376b68631106c0e1c5cbe3f385204e0
tree9c2e46d2d2ac123ba8ee7999c75805fc633c6745
parentef8e859c0b3c89846e9ceb6b292b1f9d622d6e51
[mlir] Allow for using interface class name in ODS interface definitions

It may be necessary for interface methods to process or return variables with
the interface class type, in particular for attribute and type interfaces that
can return modified attributes and types that implement the same interface.
However, the code generated by ODS in this case would not compile because the
signature (and the body if provided) appear in the definition of the Model
class and before the interface class, which derives from the Model. Change the ODS
interface method generator to emit only method declarations in the Model class
itself, and emit method definitions after the interface class. Mark as "inline"
since their definitions are still emitted in the header and are no longer
implicitly inline. Add a forward declaration of the interface class before the
Concept+Model classes to make the class name usable in declarations.

Reviewed By: rriddle

Differential Revision: https://reviews.llvm.org/D91499
mlir/test/lib/Dialect/Test/TestInterfaces.td
mlir/test/lib/IR/TestInterfaces.cpp
mlir/test/mlir-tblgen/interfaces.mlir
mlir/test/mlir-tblgen/op-interface.td
mlir/tools/mlir-tblgen/OpInterfacesGen.cpp