[PATCH] D74621: ValueTracking: Fix crash on weird size bswap

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 14 08:50:36 PST 2020


arsenm added a comment.

In D74621#1876548 <https://reviews.llvm.org/D74621#1876548>, @nikic wrote:

> I think it would make more sense to add support for this to APInt. These operations are well-defined and supported in IR, so APInt should support them as well.


I don't think this is really well defined for arbitrary odd cases. What does a bswap mean for non-byte multiples?


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

https://reviews.llvm.org/D74621





More information about the llvm-commits mailing list