[Mlir-commits] [mlir] [mlir][arith] Add `arith.fptofp` op (PR #188041)

[Renato Golin]

rengolin wrote:

> `arith.extf` / `arith.truncf` are so widely used, I'm not sure if they should be removed or deprecated. For cleanup purposes, I'd say "yes", but I'm not sure if it's worth the churn. I'm going to ask on Discourse.

I would not propose that, no.

> Maybe add a separate legalization pass which transforms `arith.fptofp -> arith.extf/truncf` and only lower `f16<->bf16`?

Wouldn't this be a canonicalization? Can be done with a folder even.

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