[PATCH] D74667: [APInt] byteSwap - handle any whole byte bitwidth greater than 16-bits

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 15 08:22:43 PST 2020


RKSimon added a comment.

In D74667#1877767 <https://reviews.llvm.org/D74667#1877767>, @arsenm wrote:

> In D74667#1877745 <https://reviews.llvm.org/D74667#1877745>, @lebedev.ri wrote:
>
> > SGTM, i'm not sure why bswap intrinsic has that limitation.
>
>
> I don't think it really does. Nothing is enforcing it, there's just a note in the LangRef


The legalizers probably assume it though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74667





More information about the llvm-commits mailing list