If the paddingAttr is an ArrayAttr with two values we know that
the element type is a ComplexType and we should pad the value
accordingly.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/lib/Dialect/Linalg/Transforms/Padding.cpp | ||
---|---|---|
131 | Is it a given that for complex types it is going to be an ArrayAttr? Seems like a fairly strong assumption. |
mlir/lib/Dialect/Linalg/Transforms/Padding.cpp | ||
---|---|---|
131 | The type is an ArrayAttr for ComplexTy but you are right, instead we should directly check the return type. Updated appropriately. |
Is it a given that for complex types it is going to be an ArrayAttr? Seems like a fairly strong assumption.