diff options
| author | Oleg Shyshkov <shyshkov@google.com> | 2022-10-13 07:28:46 +0000 |
|---|---|---|
| committer | Oleg Shyshkov <shyshkov@google.com> | 2022-10-13 07:52:43 +0000 |
| commit | c38d9cf20e7468a2618dc23fcdc66e79c925aff5 (patch) | |
| tree | 9105bb8b067d2a070a534343d8ebe8df355964ae /llvm/lib/CodeGen/ModuloSchedule.cpp | |
| parent | c5d950f4699ff0d9eee20fa144a2ef5f2deffa7b (diff) | |
| download | llvm-c38d9cf20e7468a2618dc23fcdc66e79c925aff5.zip llvm-c38d9cf20e7468a2618dc23fcdc66e79c925aff5.tar.gz llvm-c38d9cf20e7468a2618dc23fcdc66e79c925aff5.tar.bz2 | |
[mlir] Remove iterator_types() method from LinalgStructuredInterface.
`getIteratorTypesArray` should be used instead. It's a better substitute for all the current usages of the interface.
The current `ArrayAttr iterator_types()` has a few problems:
* It creates an assumption operation has iterators types as an attribute, but it's not always the case. Sometime iterator types can be inferred from other attribute, or they're just static.
* ArrayAttr is an obscure contained and required extracting values in the client code.
* Makes it hard to migrate iterator types from strings to enums ([RFC](https://discourse.llvm.org/t/rfc-enumattr-for-iterator-types-in-linalg/64535/9)).
Concrete ops, like `linalg.generic` will still have iterator types as an attribute if needed.
As a side effect, this change helps a bit with migration to prefixed accessors.
Differential Revision: https://reviews.llvm.org/D135765
Diffstat (limited to 'llvm/lib/CodeGen/ModuloSchedule.cpp')
0 files changed, 0 insertions, 0 deletions