[Mlir-commits] [mlir] [mlir][tosa][tosa-to-linalg] tosa.cast: fix answer mismatch to cast f64/f32 max value to i64/i32 (PR #130116)

[llvmlistbot at llvm.org]

ivangarcia44 wrote:

> Result overflows must be saturated.

It would be nice if TOSA cast operation matches the behavior of PyTorch and Tensorflow since they are the most popular deep learning frameworks. It would make the lowering between them smoother, less complex, and reduce the risk of bugs.

Maybe a solution to make all front ends happy could be to add an attribute to the TOSA cast operation to determine if saturation is done on overflow. This would facilitate the integration with any front end.

https://github.com/llvm/llvm-project/pull/130116