[PATCH] D108259: [AArch64][GlobalISel] Legalize scalar G_FMINNUM + G_FMAXNUM

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 17 23:10:27 PDT 2021


aemerson accepted this revision.
aemerson added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp:734-735
+      .libcallFor({s128})
+      .widenScalarToNextPow2(0)
+      .clampScalar(0, MinFPScalar, s128);
+
----------------
Is this actually needed? We don't support any non-pow-2 FP types right?


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

https://reviews.llvm.org/D108259



More information about the llvm-commits mailing list