[PATCH] D91716: AMDGPU/GlobalISel: Calculate isKnownNeverNaN for fminnum and fmaxnum

Petar Avramovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 18 08:18:12 PST 2020


Petar.Avramovic added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-fmaxnum.mir:721
     ; GFX9: [[COPY2:%[0-9]+]]:_(s32) = COPY $vgpr2
     ; GFX9: [[FCANONICALIZE2:%[0-9]+]]:_(s32) = G_FCANONICALIZE [[FMAXNUM_IEEE]]
     ; GFX9: [[FCANONICALIZE3:%[0-9]+]]:_(s32) = G_FCANONICALIZE [[COPY2]]
----------------
Thoughts on adding
```
%0:_(s32) = G_FMINNUM_IEEE/G_FMAXNUM_IEEE ...
%1:_(s32) = G_FCANONICALIZE %0
->
%0:_(s32) = G_FMINNUM_IEEE/G_FMAXNUM_IEEE ...
%1:_(s32) = COPY %0
```
combine post legalizer?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91716/new/

https://reviews.llvm.org/D91716



More information about the llvm-commits mailing list