[PATCH] D107588: [AArch64][GlobalISel] Widen G_PTRTOINT before clamping + test legalization

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 5 15:53:02 PDT 2021


paquette added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/GlobalISel/legalize-ptrtoint.mir:27
+    ; CHECK: %ptr:_(p0) = COPY $x0
+    ; CHECK: %int:_(s32) = G_PTRTOINT %ptr(p0)
+    ; CHECK: $w0 = COPY %int(s32)
----------------
aemerson wrote:
> I expected these to change?
The existing legality behaviour says it's okay:

`.legalForCartesianProduct({s1, s8, s16, s32, s64}, {p0})`

Should we just drop everything except for s64 entirely?


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

https://reviews.llvm.org/D107588



More information about the llvm-commits mailing list