Add math.erf lowering to libm call.
Add math.erf polynomial approximation.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
LGTM. Probably wait for another pair of eyes to LGTM it. @ezhulenev perhaps?
mlir/lib/Dialect/Math/Transforms/PolynomialApproximation.cpp | ||
---|---|---|
509 | nit: can you mention the source of the approximation? | |
mlir/test/Analysis/test-shape-fn-report.mlir | ||
15 ↗ | (On Diff #381147) | nit: I wouldn't add this test in this patch (it's redundant with the tanh one) |
mlir/lib/Dialect/Math/Transforms/PolynomialApproximation.cpp | ||
---|---|---|
192 | Maybe pass ArrayRef<Value> instead of a pair of iterators and a template parameter? Both call sites seems to pass values. |
Remove math.erf test from test-shape-fn-report.mlir.
Update math.erf polynomial approximation description.
Use ArrayRef in makePolynomialCalculation.
mlir/lib/Dialect/Math/Transforms/PolynomialApproximation.cpp | ||
---|---|---|
509 | I used Boost's minimax tool that utilizes the Remez method to find the constants. I will update the comment with the info. |
Maybe pass ArrayRef<Value> instead of a pair of iterators and a template parameter? Both call sites seems to pass values.