[PATCH] D107604: [AArch64][GlobalISel] Widen G_IMPLICIT_DEF and G_FREEZE before clamping

Jon Roelofs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 5 15:17:26 PDT 2021


jroelofs accepted this revision.
jroelofs added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp:83
       .legalFor(PackedVectorAllTypeList)
+      .widenScalarToNextPow2(0)
       .clampScalar(0, s8, s64)
----------------
This widen-then-clamp pattern seems really common. Maybe that ought to be a first-class operation?


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

https://reviews.llvm.org/D107604



More information about the llvm-commits mailing list