[Mlir-commits] [mlir] [Linalg] Fix SoftmaxOp's reify result shape calculation (PR #67790)
Abhishek Varma
llvmlistbot at llvm.org
Fri Sep 29 06:32:51 PDT 2023
Abhishek-Varma wrote:
> Hi @Abhishek-Varma,
>
> Could you add a test case demonstrating how the previous code was broken?
Hi @qcolombet
So, there is ambiguity involved in the naming of `LinalgOps`.
Although `SoftmaxOp` is part of `Linalg` - it is not a `linalg.generic`, it is an aggregate op.
Therefore the `cast` of the same on `LinalgOp` is wrong and crashes.
I've now demonstrated the same by adding a test case. :)
There is a better explanation [here](https://github.com/openxla/iree/pull/15021#discussion_r1336236244) by @MaheshRavishankar .
https://github.com/llvm/llvm-project/pull/67790
More information about the Mlir-commits
mailing list