| Issue |
126712
|
| Summary |
[mlir][linalg][tensor] Audit Tensor_RelayoutOp/Linalg_RelayoutOp
|
| Labels |
mlir
|
| Assignees |
|
| Reporter |
banach-space
|
As pointed out by @Groverkss in this [comment](https://github.com/llvm/llvm-project/pull/123902/files#r1927303676):
> I think DPS, tensor only ops should always be Pure. Is there a reason we have them as ConditionallySpeculatable? Just trying to understand if there is some possibility of undefined behavior i'm missing.
This is regarding [Tensor_RelayoutOp](https://github.com/llvm/llvm-project/blob/67f59a642f96cdb0b73b991b96b5ca05b988e50d/mlir/include/mlir/Dialect/Tensor/IR/TensorOps.td#L1831-L1840) that's about to be moved to Linalg:
* https://github.com/llvm/llvm-project/pull/123902/
In order to make sure that in #123902 we focus on simple code-refactor, I am re-directing the discussion on the design of `Tensor_RelayoutOp`/`Linalg_RelayoutOp` to this issue.
Thanks :)
![]()
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
