[PATCH] D115381: [AArch64][GlobalISel] Select scalar G_FMAXIMUM + G_FMINIMUM

[Jon Roelofs via Phabricator via llvm-commits]

jroelofs added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/GlobalISel/select-fmaximum.mir:43
+    ; CHECK-NEXT: %b:fpr32 = COPY $s1
+    ; CHECK-NEXT: %select_me:fpr32 = FMAXSrr %a, %b
+    ; CHECK-NEXT: $s0 = COPY %select_me
----------------
Am I reading the ARM/GISel docs correctly that G_FMAXIMUM corresponds to the FMAX instructions, G_FMAXNUM_IEEE corresponds to FMAXNM, and that there isn't direct support on ARMv8 for G_FMAXNUM?

A quick look at the SelectionDAG code suggests the opposite re: FMAXNM vs FMAXNM_IEEE though... so there's either a bug in my understanding, or SDAG got it wrong?


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

https://reviews.llvm.org/D115381