[Mlir-commits] [mlir] [MLIR][Vector] Generalize DropUnitDimFromElementwiseOps to non leading / trailing dimensions. (PR #92934)
Han-Chung Wang
llvmlistbot at llvm.org
Wed Jul 3 15:50:01 PDT 2024
hanhanW wrote:
The commit breaks downstream project. Here is the repro: `mlir-opt -test-vector-transfer-flatten-patterns ~/repro.mlir`
```mlir
func.func @unit_dim_folding(%arg0: vector<1x1xf32>) -> vector<1x1xf32> {
%cst = arith.constant dense<0.000000e+00> : vector<1x1xf32>
%0 = arith.mulf %arg0, %cst : vector<1x1xf32>
return %0 : vector<1x1xf32>
}
```
Error:
```
repro.mlir:3:8: error: 'arith.mulf' op operand #0 must be floating-point-like, but got 'vector<f32>'
%0 = arith.mulf %arg0, %cst : vector<1x1xf32>
```
The commit seems problematic for corner cases, can we revert it and re-land it with a fix?
https://llvm.org/docs/DeveloperPolicy.html#patch-reversion-policy
https://github.com/llvm/llvm-project/pull/92934
More information about the Mlir-commits
mailing list