[Mlir-commits] [mlir] [mlir][math] Propagate scalability in polynomial approximation (PR #84949)
Cullen Rhodes
llvmlistbot at llvm.org
Wed Mar 13 05:49:31 PDT 2024
================
@@ -151,6 +165,17 @@ func.func @exp_vector(%arg0: vector<8xf32>) -> vector<8xf32> {
return %0 : vector<8xf32>
}
+// CHECK-LABEL: func @exp_scalable_vector
+// CHECK-NOT: math.exp
+// CHECK-NOT: vector<8xf32>
+// CHECK: vector<[8]xf32>
+// CHECK-NOT: vector<8xf32>
+// CHECK-NOT: math.exp
----------------
c-rhodes wrote:
How about checking there are as many scalable vectors as scalars from the `@exp_scalar` test?
```suggestion
// CHECK-NOT: math.exp
// CHECK-COUNT-42: vector<[8]xf32>
// CHECK-NOT: math.exp
```
Seems a bit more useful than what it's currently doing (we know there will be a scalable vector because of the return). I understand this is based on the above test where the CHECK lines are close to useless.
https://github.com/llvm/llvm-project/pull/84949
More information about the Mlir-commits
mailing list