[PATCH] D106998: [AArch64][GlobalISel] Improve legalization for odd-sized G_ICMP/G_CONSTANT

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 28 13:13:56 PDT 2021


paquette created this revision.
paquette added a reviewer: aemerson.
Herald added subscribers: danielkiss, hiraditya, kristof.beyls, rovka.
paquette requested review of this revision.
Herald added a project: LLVM.

We were handing types like s88 like

1. clamp to the range
2. widen to the next power of 2

This isn't desirable because it causes an odd breakdown for types like s88. If we widen to the next power of 2 (s128) first, then we get a clean breakdown when we clamp back to s64.


https://reviews.llvm.org/D106998

Files:
  llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp
  llvm/test/CodeGen/AArch64/GlobalISel/legalize-cmp.mir
  llvm/test/CodeGen/AArch64/GlobalISel/legalize-constant.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106998.362512.patch
Type: text/x-patch
Size: 18159 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210728/764c0e88/attachment.bin>


More information about the llvm-commits mailing list