[llvm] SelectionDAG: ExpandFMINNUM_FMAXNUM: FCANONICALIZE is not needed for FMINNUM_IEEE (PR #139007)

YunQiang Su via llvm-commits llvm-commits at lists.llvm.org
Thu May 8 02:08:27 PDT 2025


wzssyqa wrote:

> This isn't the right sequencing. The IEEE version only exists because of the old behavior. The way to do this is to convert each target to making fminnum/fmaxnum directly legal and then deleting this expansion

Yes. I am working on it 
https://github.com/llvm/llvm-project/pull/139009
https://github.com/llvm/llvm-project/pull/139010
https://github.com/llvm/llvm-project/pull/135739

I noticed that AMDGPU also has defined FMAXNUM_IEEE. Can you take care about it since I have feel knowlege about the ISA of AMDGPU.

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


More information about the llvm-commits mailing list