[PATCH] D72492: GlobalISel: Implement lower for G_BITCAST
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 10 10:50:36 PST 2020
arsenm added a comment.
In D72492#1814575 <https://reviews.llvm.org/D72492#1814575>, @arsenm wrote:
> In D72492#1814528 <https://reviews.llvm.org/D72492#1814528>, @aemerson wrote:
>
> > On AArch64 we also use bitcast to convert from <n x p0> to <n x s64> types.
>
>
> I thought that was illegal. It’s not legal in IR
I think this is a verifier bug. You're not allowed to bitcast between scalar pointers and scalars, so allowing it for vectors doesn't make much sense. I think allowing this is potentially problematic for non-integral pointers since it makes it easy to accidentally launder away the pointerness
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72492/new/
https://reviews.llvm.org/D72492
More information about the llvm-commits
mailing list