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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 18 17:44:30 PDT 2022


efriedma added a comment.

As far as I can tell, the AArch64 fmaxnm actually corresponds to ISD::FMAXNUM_IEEE.  I wouldn't be surprised if SelectionDAG is wrong; rG687ec75d <https://reviews.llvm.org/rG687ec75d10bd860edb194d88d5438dcb1bc6eb92> was implemented well after the AArch64 backend was written.  Granted, SNaNs are rarely used, especially with floating-point exceptions disabled, so maybe nobody noticed.


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

https://reviews.llvm.org/D115381



More information about the llvm-commits mailing list