[Mlir-commits] [mlir] [mlir][arith][transforms] Adds Truncf f32 to f4e2m1 (PR #144157)

[Krzysztof Drewniak]

https://github.com/krzysz00 commented:

... I forgot I wasn't sending individual comments

I think doing extf by lookup table makes sense, and ... I honestly don't have a strong opinion about truncf, mainly because that's a series of if/select statements and not a lookup table. I could see the truncation implementation going either way

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