[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