[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