[PATCH] D115372: [AArch64][GlobalISel] Legalize scalar G_FMAXIMUM + G_FMINIMUM

Jon Roelofs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 8 12:22:06 PST 2021


jroelofs added inline comments.


================
Comment at: llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp:783
   // TODO: Vector types.
   getActionDefinitionsBuilder({G_FMAXNUM, G_FMINNUM})
       .legalFor({MinFPScalar, s32, s64})
----------------
Is it worthwhile to combine with this one, now that their legalization rules match, or do you expect they might diverge later?


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

https://reviews.llvm.org/D115372



More information about the llvm-commits mailing list