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

[Jessica Paquette via Phabricator via llvm-commits]

paquette added inline comments.


================
Comment at: llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp:83
       .legalFor(PackedVectorAllTypeList)
+      .widenScalarToNextPow2(0)
       .clampScalar(0, s8, s64)
----------------
jroelofs wrote:
> This widen-then-clamp pattern seems really common. Maybe that ought to be a first-class operation?
Yeah, I think so too. I just wanted to audit everything first.


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

https://reviews.llvm.org/D107604